-
Notifications
You must be signed in to change notification settings - Fork 6
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
Adds Gaffer support #49
base: develop
Are you sure you want to change the base?
Conversation
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.
Looks good to me - I think it'd be nice to add default paths for windows and linux as pointed out in my comment because it'd aid admins to know what to point to.
Other than that, this is indeed what is needed to add gaffer app support.
Might be nice in the PR description to point to a matching ayon-gaffer integration addon repository that may work with this PR - just so it's tracked here for the future.
E.g. I know of https://github.com/RVXStudio/ayon-gaffer
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.
I've pushed a minimal host addon now |
Changelog Description
Added support for Gaffer and should work with ayon-gaffer addon
https://github.com/RVXStudio/ayon-gaffer
Additional review information
https://github.com/GafferHQ/gaffer
Testing notes: