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

Update CONTRIBUTING.md with link to discord #5311

Closed
wants to merge 4 commits into from
Closed

Update CONTRIBUTING.md with link to discord #5311

wants to merge 4 commits into from

Conversation

carlosm-o
Copy link

I had an issue with running the application locally and couldn't find where to go for live help other than submitting an issue. I hope this can help other potential contributors reach out to the Cypress community.

@netlify
Copy link

netlify bot commented Jun 15, 2023

👷 Deploy request for cypress-docs pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 02e4cbc

@CLAassistant
Copy link

CLAassistant commented Jun 15, 2023

CLA assistant check
All committers have signed the CLA.

@cypress-app-bot
Copy link
Collaborator

@MikeMcC399
Copy link
Contributor

MikeMcC399 commented Jun 15, 2023

@carlosm-o

How about putting a link to https://www.cypress.io/support/ into the footer of https://docs.cypress.io/ ?

image
Edit: suggested in PR #5331

You shouldn't be changing package-lock.json in this PR.

Copy link
Contributor

@nagash77 nagash77 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you please fix the package.lock file? @MikeMcC399 is correct there is no reason this should be updated in this PR.

@nagash77 nagash77 self-assigned this Jun 15, 2023
Comment on lines +266 to +268
We have a thriving community on discord. If you have any questions related to
contributing please reach out, we'd be happy to help!
[Join us on discord to start the conversation](https://discord.gg/cypress).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
We have a thriving community on discord. If you have any questions related to
contributing please reach out, we'd be happy to help!
[Join us on discord to start the conversation](https://discord.gg/cypress).
We have a thriving community on Discord. If you have any questions related to
contributing please reach out, we'd be happy to help!
[Join us on Discord to start the conversation](https://discord.gg/cypress).

Discord is written uppercase.

There are different links in use for Cypress on Discord:

https://on.cypress.io/discord

is also used in https://github.com/cypress-io/cypress/blob/develop/.github/ISSUE_TEMPLATE/1-bug-report.yml

The support page lists

https://discord.com/invite/cypress

This should be checked, which is the preferred Discord link to be passed on.

Copy link
Member

@emilyrohrbough emilyrohrbough Jun 15, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The discord channel is not monitored for helping individuals contribute to Cypress. It would be more appropriate to remove these changes to the Contribution guide & add the Discord chat badge to the README. See: https://github.com/cypress-io/cypress#readme

Also the best link to use is on.cypress.io/chat. This is managed by our team to direct users to the correct invite if/when it expires.

Copy link
Contributor

@MikeMcC399 MikeMcC399 Jun 15, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@emilyrohrbough

I was a bit confused about "It would be more appropriate to remove add the Discord chat badge to the README. See: https://github.com/cypress-io/cypress#readme". Did you mean "remove" or "add"?

The README of the main Cypress repo (not the docu repo) contains

Discord chat

which links to https://on.cypress.io/discord

Are you saying that the link in the README should be changed to
https://on.cypress.io/chat
instead?

Copy link
Member

@emilyrohrbough emilyrohrbough Jun 15, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry - updated comment. but to clarify - I am suggesting removing the suggested changes to the contribution guide all together and adding the badge to the readme instead.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@emilyrohrbough

I am suggesting removing the suggested changes to the contribution guide all together and adding the badge to the readme instead.

I agree that adding this information to CONTRIBUTING.md is the wrong place. The discussions which take place in Discord are about using Cypress. I have not seen discussions there about contributing to the development of Cypress in terms of PRs to the software or to the documentation.

The README in this (documentation) repo could copy the badge used in the README of the main repo.

Additionally (and probably outside of the scope of this PR), there could be a section "How to get support" added to

each linking to the detailed webpage https://www.cypress.io/support/ how to get support.

Copy link
Contributor

@MikeMcC399 MikeMcC399 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please change Discord to uppercase.

Comment on lines +266 to +268
We have a thriving community on discord. If you have any questions related to
contributing please reach out, we'd be happy to help!
[Join us on discord to start the conversation](https://discord.gg/cypress).
Copy link
Member

@emilyrohrbough emilyrohrbough Jun 15, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The discord channel is not monitored for helping individuals contribute to Cypress. It would be more appropriate to remove these changes to the Contribution guide & add the Discord chat badge to the README. See: https://github.com/cypress-io/cypress#readme

Also the best link to use is on.cypress.io/chat. This is managed by our team to direct users to the correct invite if/when it expires.

@nagash77
Copy link
Contributor

@emilyrohrbough I disagree that discord cannot help folks contribute. Lots of folks in there have experience contributing to the repo. So I think it is fine to point folks there for assistance.

I do like the idea of using the discord badge.

@MikeMcC399
Copy link
Contributor

@MikeMcC399
Copy link
Contributor

@carlosm-o
Are you intending to respond to the comments for change?

@MikeMcC399
Copy link
Contributor

@mike-plummer
Copy link
Contributor

@carlosm-o Thanks so much for the contribution! It sparked a good discussion and we now have a link to Discord via a badge on the README from #5335. Since this should give folks a good route to the community I'm going to close this PR - lmk if you disagree and we can reopen for further discussion.

@nagash77 nagash77 removed their assignment Jul 7, 2023
@carlosm-o carlosm-o deleted the add-discord-link-to-contributing-file branch August 16, 2023 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants