Skip to content

Cherrypick main into dev 2.0 - Contributor docs and all-contributors only #7730

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

Closed
wants to merge 13 commits into from
377 changes: 363 additions & 14 deletions .all-contributorsrc

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions .github/first-timers.yml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# You can create the issue in a different repo than where the problem is. Just make sure you installed the bot on the configured repository.
# You can create the issue in a different repo than where the problem is. Just make sure you have installed the bot on the configured repository.
# The issue will link back to the original repository where the contribution will be made.
repository: p5.js

@@ -11,4 +11,4 @@ labels:
- hacktoberfest

# If you would like to add your own template for the issue, add an .md file to your .github folder
# template: .github/first-timers-issue-template.md
# template: .github/first-timers-issue-template.md
77 changes: 61 additions & 16 deletions README.md

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion contributor_docs/archive/es6-adoption.md
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
## ES6 Adoption
p5.js has recently adopted the ECMAScript 2015 (ES6) language specifications in order to reduce the complexity of the codebase, increase readability, and utilize features that facilitate elegant and effective coding practices for both newcomers and seasoned contributors.

The transition process was initially [discussed](https://github.com/processing/p5.js/issues/3758) with the aim of migrating p5 to ES6, which later lead to a series of widespread syntactical changes in the codebase aligned with ES6 specifications. More info on the initial transition can be found [here](https://github.com/processing/p5.js/pull/3874). These changes consisted of minor modifications to the build system to facilitate processing, linting and testing the library based on ES6 standards, as well as major and ubiquitous syntactical modification in line with ES6 features.
The transition process was initially [discussed](https://github.com/processing/p5.js/issues/3758) with the aim of migrating p5 to ES6, which later led to a series of widespread syntactical changes in the codebase aligned with ES6 specifications. More info on the initial transition can be found [here](https://github.com/processing/p5.js/pull/3874). These changes consisted of minor modifications to the build system to facilitate processing, linting and testing the library based on ES6 standards, as well as major and ubiquitous syntactical modification in line with ES6 features.

It is worthy to note that as of writing this, these transformations are by no means complete, and do not reflect nor implement every possible feature of ES6. They are intended to facilitate a smoother transition to properly and efficiently utilize ES6 features if and when aligned with the community interests and standards. And serve to motivate contributors to gradually conform to the new style and features.

2 changes: 1 addition & 1 deletion contributor_docs/how-to-add-friendly-error-messages.md
Original file line number Diff line number Diff line change
@@ -163,7 +163,7 @@ Next, you will file an issue ticket to discuss creating a new case or confirm yo

Go to the [issue board](https://github.com/processing/p5.js/issues), press the "New Issue" button, and then choose the "Issue: 💡 Existing Feature Enhancement" option. An empty form should appear.

Add a title along the lines of “Adding a new case to `fileLoadErrrorCases`: \[a high-level description of your file load error case].” For the “Increasing access” section, enter a short paragraph on the typical scenario you prepared at the beginning of this step.
Add a title along the lines of “Adding a new case to `fileLoadErrorCases`: \[a high-level description of your file load error case].” For the “Increasing access” section, enter a short paragraph on the typical scenario you prepared at the beginning of this step.

Then, check the “Friendly Errors” box for the “Most appropriate sub-area of p5.js?” question. Lastly, for the “Feature enhancement details” section, enter your paragraph detailing your error handling and what file types it loads.

Loading