-
Notifications
You must be signed in to change notification settings - Fork 25
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
chore: remove flavor-related sanity checks #245
Conversation
Is the idea to remove flavors completely? Then I'd also remove the configuration option, and add deprecation warnings to the functions (with instructions to set the ROCK_SELECTED_FLAVOR manually, if required) https://github.com/rock-core/package_set/blob/master/init.rb#L61-L142 |
Rather obviously, since 'stable' is pre-2016 I think ... or even earlier.
As I said. I don't have time to do the whole removal. If you volunteer, be my guest. |
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.
Looks good to me.
Do we actually need deprecation? I think we could also just remove the functionality, as it wasn't used for a long time. |
Created a PR to this branch to also remove leftover stuff: #246 |
Agreed. I did not add deprecations... |
I don't have the time to remove it completely, but that part is safe enough.