-
Notifications
You must be signed in to change notification settings - Fork 139
Fix typos in Examples subdirectory #216
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
Conversation
Found via codespell Signed-off-by: luzpaz <[email protected]>
Examples/ColourSpace/colourspace.cpp
Outdated
| gEffectHost->getPropertySet(effect, &effectProps); | ||
|
|
||
| // retieve the colour management style that the host has decided we shall use | ||
| // reteive the colour management style that the host has decided we shall use |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be "retrieve" (and below)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
Sorry, one more thing @luzpaz — all commits must have a "Signed-Off-By: " line at the end so we can track contributions for legal reasons. Could you redo your commit with a force-push? |
Signed-off-by: luzpaz <[email protected]>
Found via codespell Signed-off-by: luzpaz <[email protected]>
Found via codespell Signed-off-by: luzpaz <[email protected]>
AKA the Open Effects Association. Signed-off-by: Gary Oberbrunner <[email protected]>
Also update the doc build script to use `uv` if present. That's nicer because you don't have to manually create a python venv or install depencies. Removed old unused footer, and marked ofx-header and ofx-footer as unused. Signed-off-by: Gary Oberbrunner <[email protected]>
ab87c05 to
a67a271
Compare
|
I hope I did the rebase and amend correctly 🤞 |
Signed-off-by: Gary Oberbrunner <[email protected]>
Signed-off-by: Gary Oberbrunner <[email protected]>
|
Hmm, somehow the new version pulled in a bunch of other stuff. No worries, I pulled your diff into a new PR and applied it. Thanks very much for your contributions! |
Found via codespell