-
-
Notifications
You must be signed in to change notification settings - Fork 1
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
modulesync #1
modulesync #1
Conversation
failing tests will be fixed by #3 |
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.
Probably fine. Please rebase on top of master to re-trigger CI when the other PRs are merged to check if the tests are green.
Since these change seems to fix a module that is not currently working, maybe we don't need PR for each change and fixes can be added on top of this branch.
.fixtures.yml
Outdated
systemd: "puppet/systemd" | ||
stdlib: "puppetlabs/stdlib" |
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.
Please use links to git repos instead:
fixtures:
repositories:
stdlib: https://github.com/puppetlabs/puppetlabs-stdlib.git
systemd: https://github.com/voxpupuli/puppet-systemd.git
We test against the main branches of our dependencies to detect breaking changes before they are released. We use forge releases within our acceptance tests.
Ready for another review |
First module sync on new module