-
Notifications
You must be signed in to change notification settings - Fork 614
drop code for Debian 6/7 and Ubuntu 10.04/12.04 #1290
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
Conversation
postgresql::globals is a classBreaking changes to this file WILL impact these 19 modules (exact match):
Breaking changes to this file MAY impact these 2 modules (near match):postgresql::server::config_entry is a typeBreaking changes to this file WILL impact these 6 modules (exact match):Breaking changes to this file MAY impact these 1 modules (near match):postgresql::server::initdb is a classthat may have no external impact to Forge modules. This module is declared in 70 of 578 indexed public
|
bfcf178
to
cde381d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make sense!
The test failures look unrelated to this change. |
Test failures should be fixed with 09b242a. Mind rebasing. |
cde381d
to
ca4912a
Compare
rebased |
Test failures are relevant. Sadly the tests in this module are quite dated. |
ca4912a
to
8cc1660
Compare
Updated the tests from D6 to D10 (there is no D11 for reasons ;)) |
#1288 should add D11, but there's some CI issue. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
@david22swan any idea why CentOS 8 is failing? I can only see that it's timing out but is there a log of the actual run? |
drop code for Debian 6/7 and Ubuntu 10.04/12.04
Support for these was dropped in 3d1e1e3 and e61579f, but the code
remained.