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
Links to sign the CLA are broken/misleading #9133
Comments
Yowzer, I thought we had fixed all of those. Thank you for the issue! I think we'll just mention that a link will come up, because I've seen issues when people just go straight to the CLA page without having first come from a pull request. |
@Standard8 Would you like to make a PR? We'd love the help! |
Likewise, the "Contributing Code" section in eslint/CONTRIBUTING.md is similarly misleading. Anyone mind if I take on this issue? |
@i-ron-y Go for it! Feel free to ping back here or to stop by the ESLint Gitter with any questions you might have. Thanks for taking this on! |
Hmm... as the Pull Request Guidelines page links to https://cla.js.foundation/eslint/eslint, to keep it all consistent, I decided to have all mentions of the CLA link to https://cla.js.foundation/eslint/eslint... But I'm just as happy to revise all three pages to omit said link and instead inform the potential contributor that a link to sign the CLA will appear when they submit a pull request, if that's the preferred option. |
I think you need to give a people a link to the CLA before they start submitting, in case they disagree to it for some reason. The current link just didn't work with the text - it said sign here, but then there was nothing to sign. The PR bot that checks it, is then just a backup in case they didn't read everything. |
This issue has been fixed with PR #9232 closing. |
I was preparing to submit a PR and trying to follow the documentation. I came across this page first:
https://eslint.org/docs/developer-guide/contributing/
which said I should sign the CLA, and linking to https://contribute.jquery.org/cla - that was confusing as it was saying I should enter my details but didn't tell me where.
It should either link to https://cla.js.foundation/eslint/eslint, or be reworded to say that you'll get a link to sign it when you submit the PR.
The text was updated successfully, but these errors were encountered: