-
Notifications
You must be signed in to change notification settings - Fork 7
Remove GitHub
prefix on ERB lint rules
#4
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
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.
IMO, I agree the rename could attract more contributors outside of GitHub and be an accelerator for this set of linting rules to grow and become more robust. 👍
Definitely agree with @inkblotty, I like the idea of more open-source style without the GitHub prefix. |
👋🏻 thanks for the ping @khiga8! I'm torn on this idea. On one hand, it does make sense to me given the current linters we've shipped. However, we do have several non-a11y linters we could open-source into this library and I worry that we'd be less likely to do so if we renamed it. What if we moved towards contributing to |
I think that this library can be a11y only. It would bring a lot of value to the community and reach a broader user base |
Thank you all for the input! I really appreciate it :)
That is a good point. I think that would be a good eventual end-goal, but I might prefer us taking more ownership of these rules to start with. This would allow us to work more quickly and give greater input into the rulesets that are being introduced. Given our team's AOR, I feel that our team might be well-equipped to give input on the rulesets being introduced. What do you think? @joelhawksley |
If we're confident enough to open-source these rules as we have already, I think it's a good enough sign that they could be proposed upstream for |
I don't think we want to depend on them to release new linters. From my last conversation with them, it looks like the projects is maintained by a single person and from my experience it takes some time for the PRs to be reviewed/merged. I think that having an a11y gem is a good idea so we can ship fast and also bring value to the community. At some point, I do think it's worth exploring some kind of partnership with Shopify so we can help maintain erblint |
@manuelpuyol that's a good point. It would be good to be able to control releases. Have you asked about gaining publish rights on Regardless, I think I see the point now. I'm 👍🏻 to rename. |
Thanks again everyone! There are some convos taking place next week that could affect trajectory of this convo so I'll hold off on naming until after the convos take place. I'll close this PR for now. I'll open a separate issue to follow up on the renaming. |
This PR removes the
GitHub
prefix on rules because it is unnecessary/overly verbose.Discussion
I was discussing this idea with @manuelpuyol, and we think it might make sense to rename this gem and repo to
erblint-a11y
. The reason is because it may be much easier to gain attention/contributions from other folks interested in Rails/ERB accessibility tools. It may also be easier to collab on this with other Rails teams (like Shopify who open sourced erb-lint).Most of our ERB lint rules are a11y focused anyways so I am not sure how likely it is we'll open source other rules.
If we do rename this gem/repo, I think we'll want to it in the next-next release after adding a deprecation warning. Not totally sure what the best way to go about it would be.
Thoughts on this idea? cc: @joelhawksley , @github/accessibility-reviewers