Skip to content

Support using git addon version in makefile manage #114

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

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

Conversation

MustafaJafar
Copy link
Member

@MustafaJafar MustafaJafar commented Aug 11, 2025

Changelog Description

An alternative to #94

This PR adds support for AYON_USE_GIT_SUFFIX_FOR_ADDONS env var for initializing the processor services with addon version with the +git suffix. This is the same env var used on the ayon-docker server.

AYON_SERVER_URL=http://localhost:5000/
AYON_API_KEY=veryinsecureapikey
AYON_USE_GIT_SUFFIX_FOR_ADDONS=True

Additional details

when working with kitsu addon development, it's possible to develop from within the ayon-docker directly or from a separate directory.

Although, developing from within the ayon-docker/addons directly, if someone does, running servies live from source code won't work becasue the service is not initialized to work with the +git vesion. so, it fails to fetch settings.

I think such a solution can be adapted in other addons with services.

Testing notes:

  1. when setting AYON_USE_GIT_SUFFIX_FOR_ADDONS to true it should set the addon version to +git.
  2. when no setting it or setting it to false, the actual addon version should be used.

@MustafaJafar MustafaJafar self-assigned this Aug 11, 2025
@MustafaJafar MustafaJafar added the type: enhancement Improvement of existing functionality or minor addition label Aug 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement Improvement of existing functionality or minor addition
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant