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

Clone p5.js's latest release instead of main, and remove old reference items #553

Merged
merged 6 commits into from
Sep 18, 2024

Conversation

davepagurek
Copy link
Collaborator

@davepagurek davepagurek commented Sep 17, 2024

Resolves #522

This makes two changes to builder scripts:

  • A little update to make it easier to make updates to the reference builder (the thing npm run build:reference calls.) Currently, it just clones the p5.js repo's main branch, which is ever changing, but we only want to publish the current release's info on the site. Now, we can rebuild the reference as much as we want without worrying about "leaking" new APIs before they're released.
  • Makes it remove all the reference items before building again. Previously, it would overwrite existing files or create new files where they do not yet exist, but it would never delete things that are no longer in the reference. This resolves Problem with html symbols at https://p5js.org/reference/#Foundation #522 because that item simply is no longer in the reference.

@davepagurek davepagurek changed the title Clone p5.js's latest release instead of main Clone p5.js's latest release instead of main, and remove old reference items Sep 18, 2024
@Qianqianye Qianqianye merged commit 66a3813 into main Sep 18, 2024
4 checks passed
@Qianqianye Qianqianye deleted the clone-release branch September 18, 2024 05:14
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.

Problem with html symbols at https://p5js.org/reference/#Foundation
2 participants