-
Notifications
You must be signed in to change notification settings - Fork 9
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
Add resolved rules and comments. #2
Conversation
70fa7f0
to
dc216b6
Compare
Have you already reached out to Carson/David/Tim to get their feedback on this PR even if they're not yet members of the /ni org? |
The link is now available in the channel. |
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.
Overall, great! I would like to see the rationale brought out of our internal PR and into the open here, since this repo is the style guide, and not my random internal PR which will be long forgotten next year.
dc216b6
to
0f93088
Compare
Notes
Linted and tested