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

Fixed error in references #5251

Merged
merged 4 commits into from
May 24, 2021
Merged

Fixed error in references #5251

merged 4 commits into from
May 24, 2021

Conversation

aLyonsGH
Copy link
Contributor

Addresses #2865
In the description of lerpColor(), I changed "the two values where 0.0 equal to the first color" to "the two values where 0.0 is equal to the first color" (I added the "is")

@welcome
Copy link

welcome bot commented May 16, 2021

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

@aLyonsGH aLyonsGH changed the title Add files via upload Fixed error in references May 16, 2021
@limzykenneth
Copy link
Member

Hi @aLyonsGH, I think you have accidentally saved the edited file as a new file at the root of the project folder instead of saving over the original file. Can you try can revert the current changes and save the changes in the original file? Thanks.

aLyonsGH added 2 commits May 17, 2021 18:11
Fixed file accidentally added to root, now fix is added here on line 364
Oops! Moved the change to the proper file location.
@aLyonsGH
Copy link
Contributor Author

Done! I think this is all set now? Thank you for your patience, this is my first contribution to p5.js!

@limzykenneth
Copy link
Member

I'm not sure why the CI tests are not running, they keep getting cancelled after starting for a few minutes. @outofambit Do you have an idea what's going on?

Copy link
Contributor

@outofambit outofambit left a comment

Choose a reason for hiding this comment

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

looks good to me! i re-ran the CI jobs manually, will merge when they finish (i'm sure it'll be fine)

@outofambit outofambit merged commit 8764b25 into processing:main May 24, 2021
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