Skip to content
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

feat: manage timesyncd package on Debian >= 11 and Ubuntu >= 20.04 #296

Merged
merged 1 commit into from
Sep 28, 2022

Conversation

saz
Copy link
Sponsor Contributor

@saz saz commented Sep 28, 2022

Fixes #294

Copy link
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm leaning to disabling RSpec/RepeatedExample to (at least locally) to make Rubocop happy. I don't mind the repetition here.

@saz
Copy link
Sponsor Contributor Author

saz commented Sep 28, 2022

@ekohl I've just pushed a change removing the repeated examples, although I'd say it's better to keep them, just to be sure.

spec/classes/init_spec.rb Outdated Show resolved Hide resolved
@ekohl ekohl merged commit 264b400 into voxpupuli:master Sep 28, 2022
@bastelfreak bastelfreak added the enhancement New feature or request label Sep 28, 2022
@jhoblitt jhoblitt mentioned this pull request Jan 26, 2023
@bastelfreak
Copy link
Member

@jhoblitt while I'm the person that added the enhancement label, I now think bug is better. Debian 10/Ubuntu 20.04 was already in metadata.json, so one could argue this PR doesn't add a new feature/support, it fixes existing code.

@bastelfreak bastelfreak added bug Something isn't working and removed enhancement New feature or request labels Jan 27, 2023
@jhoblitt
Copy link
Member

@bastelfreak copy. Thank you. I will reroll the changelog.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

systemd-timesyncd package should be managed, if required
4 participants