-
Notifications
You must be signed in to change notification settings - Fork 466
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
[ruler] Remove unused fields #2288
Conversation
a404d73
to
1211a34
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.
Thanks! Could you add a [CHANGE]
CHANGELOG entry please?
I was helping someone debug notifications being sent every min and we found that these two fields are not used anywhere at all. I am not sure of the deprecation and breaking change policy. Should I mark them deprecated somehow and then remove them or just remove them right away since they are not being used. Signed-off-by: Goutham Veeramachaneni <gouthamve@gmail.com>
1211a34
to
44d696a
Compare
Done! |
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.
LGTM if we're OK just removing these fields. Since they're unused and no longer do anything, I can't imagine anyone has them set?
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.
Change lgtm. Not sure about the deprecation notice process either though.
Yeah, I wouldn't go through the deprecation policy given they're unused. |
* [ruler] Remove unused fields I was helping someone debug notifications being sent every min and we found that these two fields are not used anywhere at all. I am not sure of the deprecation and breaking change policy. Should I mark them deprecated somehow and then remove them or just remove them right away since they are not being used. Signed-off-by: Goutham Veeramachaneni <gouthamve@gmail.com> * Update CHANGELOG.md Co-authored-by: Marco Pracucci <marco@pracucci.com>
I was helping someone debug notifications being sent every min and we found that
these two fields are not used anywhere at all.
I am not sure of the deprecation and breaking change policy. Should I mark them
deprecated somehow and then remove them or just remove them right away since they
are not being used.
Checklist
CHANGELOG.md
updated - the order of entries should be[CHANGE]
,[FEATURE]
,[ENHANCEMENT]
,[BUGFIX]