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: deprecate personal config (fixes #11914, refs eslint/rfcs#32) #12426
Conversation
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.
Just requesting two small wording changes, otherwise this looks good to me. Thanks!
Co-Authored-By: Kevin Partington <platinum.azure@kernelpanicstudios.com>
Co-Authored-By: Kevin Partington <platinum.azure@kernelpanicstudios.com>
Thank you for review. I have applied your suggestions. |
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 for working on this! Left a few suggestions.
Co-Authored-By: Kai Cataldo <7041728+kaicataldo@users.noreply.github.com>
Co-Authored-By: Kai Cataldo <7041728+kaicataldo@users.noreply.github.com>
Co-Authored-By: Kai Cataldo <7041728+kaicataldo@users.noreply.github.com>
Co-Authored-By: Kai Cataldo <7041728+kaicataldo@users.noreply.github.com>
I have updated this PR to follow the review. |
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, thanks!
@platinumazure Mind giving this one more look to confirm your concerns have been addressed? |
I apologize, I had forgotten about this one. Thanks @mysticatea and @kaicataldo for your patience! |
What is the purpose of this pull request? (put an "X" next to item)
[X] Documentation update
[X] Other, please explain: deprecate a core feature
What changes did you make? (Give an overview)
This PR soft-deprecates the personal config file. This decision happened on RFC32 after RFC28's discussion. I removed the scattered descriptions about the personal config file then added those as the last section. Therefore, people can know what is deprecated clearly.
This is documentation-only change, but the label of this PR is
Update:
because deprecation should be semver-minor.Is there anything you'd like reviewers to focus on?
Nothing in particular.