Update CONTRIBUTING.md links and add Discord invitation #8190
Conversation
- Update contributor docs link to https://p5js.org/contribute/ - Update community statement link to https://p5js.org/about/#community-statement - Add Discord server link with #contribute-to-p5 channel mention Fixes processing#6929
|
🎉 Thanks for opening this pull request! For guidance on contributing, check out our contributor guidelines and other resources for contributors! Thank You! |
|
Thanks for spotting the issue and addressing it, @RishiAhuja ! I'll merge this shortly. In the future, please ask to be assigned on the issue before filing the PR, especially in more complex cases where technical consensus is needed before work can go ahead (the "Introduction" section of the contributor guideline has a bit more about this). |
|
Thanks for merging, @ksen0, I appreciate it! And thanks for the feedback regarding assignment. My thinking was that since the issue was inactive for a while after your comment detailing the exact link updates needed, and it was a straightforward documentation change. But I understand the importance of signaling intent first, especially for coordination on bigger tasks. I'll be sure to comment and ask to be assigned next time before starting work. Looking forward to contributing more! |
2 participants
Fixes #6929
Changes:
contributor docslink inCONTRIBUTING.mdto point to the correct page:https://p5js.org/contribute/.community statementlink to point to the correct section on the website:https://p5js.org/about/#community-statement.#contribute-to-p5channel as suggested in the issue.These changes address the outdated links identified in issue #6929, making the contribution guide more accurate and helpful for new contributors.
Screenshots of the change:
N/A (Text-only change in Markdown file)
PR Checklist
npm run lintpasses (Make sure to runnpm run lintlocally. If it passes, check this box[x])