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

Add JSON file for PVector#setHeading() #621

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

babaissarkar
Copy link

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
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
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
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