Skip to content

Fixed minor error in documentation for p5.Vector.sub()#7265

Merged
davepagurek merged 1 commit into
processing:mainfrom
jaredberghold:vector-sub-documentation-edit
Sep 23, 2024
Merged

Fixed minor error in documentation for p5.Vector.sub()#7265
davepagurek merged 1 commit into
processing:mainfrom
jaredberghold:vector-sub-documentation-edit

Conversation

@jaredberghold
Copy link
Copy Markdown
Contributor

@jaredberghold jaredberghold commented Sep 14, 2024

Quick and simple edit (no issue created)

Changes:

Fixed minor error in the documentation for Vector sub() to make it clearer that p5.Vector.sub() does not modify the original vectors. Specifically, changed a comment in one of the code examples from "// Subtract with modifying the original vectors." to "// Subtract without modifying the original vectors."

Screenshots of the change:

PR Checklist

Fixed minor error in the documentation for Vector sub() to make it clearer that p5.Vector.sub() does not modify the original vectors.
@welcome
Copy link
Copy Markdown

welcome Bot commented Sep 14, 2024

🎉 Thanks for opening this pull request! Please check out our contributing guidelines if you haven't already. And be sure to add yourself to the list of contributors on the readme page!

Copy link
Copy Markdown
Collaborator

@perminder-17 perminder-17 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, looks good to me. :)

@davepagurek davepagurek merged commit 1d8ebf3 into processing:main Sep 23, 2024
@davepagurek
Copy link
Copy Markdown
Contributor

@all-contributors please add @jaredberghold for docs

@allcontributors
Copy link
Copy Markdown
Contributor

@davepagurek

I've put up a pull request to add @jaredberghold! 🎉

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.

3 participants