-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Update finish scripts for s6 v3 #2555
Conversation
b7029c0
to
715f07f
Compare
@pvizeli After this one is merged I'll do another PR pass to add codenotary to all the ones missing it. But I did add it to the 2 with the version being bumped now since that makes sense. |
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, @mdegat01 👍
I think this looks good and can be merged (if we follow up with the codenotary change as suggested by Pascal in follow-up PR).
* Bump version for addons with recent updates * Use exec and add codenotary * Codenotary not a boolean
Update finish scripts for https://developers.home-assistant.io/blog/2022/05/12/s6-overlay-base-images
For most addons did not bump the version, just ensuring the next PR to go in doesn't have a surprise issue. Did bump the version for ZwaveJS and Deconz since they have all had recent updates so want to make new images without it.