Skip to content
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

Closed
Standard8 opened this issue Aug 21, 2017 · 7 comments
Closed

Links to sign the CLA are broken/misleading #9133

Standard8 opened this issue Aug 21, 2017 · 7 comments
Labels
accepted There is consensus among the team that this change meets the criteria for inclusion archived due to age This issue has been archived; please open a new issue for any further discussion documentation Relates to ESLint's documentation good first issue Good for people who haven't worked on ESLint before

Comments

@Standard8
Copy link
Contributor

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.

@eslintbot eslintbot added the triage An ESLint team member will look at this issue soon label Aug 21, 2017
@platinumazure
Copy link
Member

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.

@platinumazure platinumazure added accepted There is consensus among the team that this change meets the criteria for inclusion documentation Relates to ESLint's documentation and removed triage An ESLint team member will look at this issue soon labels Aug 21, 2017
@kaicataldo
Copy link
Member

@Standard8 Would you like to make a PR? We'd love the help!

@ilyavolodin ilyavolodin added the good first issue Good for people who haven't worked on ESLint before label Sep 3, 2017
@i-ron-y
Copy link
Contributor

i-ron-y commented Sep 5, 2017

Likewise, the "Contributing Code" section in eslint/CONTRIBUTING.md is similarly misleading.

Anyone mind if I take on this issue?

@kaicataldo
Copy link
Member

@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!

@i-ron-y
Copy link
Contributor

i-ron-y commented Sep 5, 2017

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.

@Standard8
Copy link
Contributor Author

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.

@ilyavolodin
Copy link
Member

This issue has been fixed with PR #9232 closing.

@eslint-deprecated eslint-deprecated bot locked and limited conversation to collaborators Mar 5, 2018
@eslint-deprecated eslint-deprecated bot added the archived due to age This issue has been archived; please open a new issue for any further discussion label Mar 5, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
accepted There is consensus among the team that this change meets the criteria for inclusion archived due to age This issue has been archived; please open a new issue for any further discussion documentation Relates to ESLint's documentation good first issue Good for people who haven't worked on ESLint before
Projects
None yet
Development

No branches or pull requests

6 participants