Skip to content

Add JSON file for PVector#setHeading() #621

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

Merged
merged 3 commits into from
Jul 11, 2025

Conversation

babaissarkar
Copy link
Contributor

Resolves #620 by adding the JSON file for PVector#setHeading() generated using the instruction at https://github.com/processing/processing-website?tab=readme-ov-file#keywordstxt.

Copy link

netlify bot commented Mar 26, 2025

Deploy Preview for java-processing-faf822 ready!

Name Link
🔨 Latest commit 8018e7e
🔍 Latest deploy log https://app.netlify.com/sites/java-processing-faf822/deploys/67e3c0db16697b00085ed848
😎 Deploy Preview https://deploy-preview-621--java-processing-faf822.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@babaissarkar
Copy link
Contributor Author

babaissarkar commented Mar 26, 2025

Looks like the deploy preview is missing setHeading(), is anything still missing?
Edit: how do I see all pages that are generated? ...59 more pages

@SableRaf
Copy link
Collaborator

SableRaf commented Mar 26, 2025

Hi @babaissarkar

The .json files should be generated by the Doclet script:

https://github.com/processing/processing-doclet/blob/main/README.md

The instructions are a bit outdated (still mentioning Ant) but the overall process should work to generate the .json files.

Note: The fact that it requires cloning multiple repos next to each other on your local machine, running a script, and pushing the changes is a holdover from the old days of the project and not a recommended practice in general. We'll have to automate this with GitHub actions at some point.

@babaissarkar
Copy link
Contributor Author

babaissarkar commented Mar 26, 2025

The .json files should be generated by the Doclet script:

I generated them with the Doclet script, but forgot to add PVector.json and the updated one for PVector#heading().

Note: The fact that it requires cloning multiple repos next to each other on your local machine, running a script, and pushing the changes is a holdover from the old days of the project and not a recommended practice in general. We'll have to automate this with GitHub actions at some point.

I agree, this should be automated.

@SableRaf SableRaf self-requested a review March 26, 2025 10:48
@SableRaf SableRaf merged commit d8fc541 into processing:main Jul 11, 2025
4 checks passed
@SableRaf
Copy link
Collaborator

SableRaf commented Jul 11, 2025

Merged. Thanks for your patience! ✨

@babaissarkar babaissarkar deleted the update_set_heading_json branch July 11, 2025 15:01
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.

PVector.setHeading() reference page missing
2 participants