-
Notifications
You must be signed in to change notification settings - Fork 183
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for Debian 12 and Ubuntu 24.04 #428
Conversation
Can confirm that this is required and works as expected. Otherwise the global default lead to this error:
@ghoneycutt Can we please get this merged and released? :) |
This data is valid, so people can copy it and use it today before it is merged as this module is forward compatible in that way. |
@flichtenheld could you please add the acceptance testing to support this? |
Will do |
@ghoneycutt Added enough to make |
7d9b90c
to
3b3bdd8
Compare
Simple copy of Debian 11 and Ubuntu 22.04 config. Saw no reason to change any of the defaults. Definitely better than the global defaults... Fixes: ghoneycutt#427 Signed-off-by: Frank Lichtenheld <[email protected]>
Signed-off-by: Frank Lichtenheld <[email protected]>
Now also enabled GHA tests and rebased on newest main. |
Approved the tests and they are coming up with errors. |
Thanks @baurmatt and @flichtenheld ! |
Simple copy of Debian 11 and Ubuntu 22.04 config. Saw no reason to change any of the defaults. Definitely better than the global defaults...
Fixes: #427