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

Chore: Use folder in cli arguments #197

Merged
merged 4 commits into from
Apr 3, 2024

Conversation

iLLiCiTiT
Copy link
Member

@iLLiCiTiT iLLiCiTiT commented Mar 19, 2024

Changelog Description

Use --folder instead of --asset argument in command line interface.

Additional info

Use folder over asset argument in command line interface. Modified royal render and deadline to support new argument. Deadline plugin does support both options, tries to use --asset if first attempt fails. This PR affects only command selectcontext used in royal render.

@ynbot ynbot added module: Deadline module: RoyalRender size/XS type: enhancement Improvement of existing functionality or minor addition labels Mar 19, 2024
@iLLiCiTiT iLLiCiTiT added Backwards incompatible and removed size/XS type: enhancement Improvement of existing functionality or minor addition module: Deadline module: RoyalRender labels Mar 19, 2024
@ynbot ynbot added module: Deadline module: RoyalRender type: enhancement Improvement of existing functionality or minor addition labels Mar 19, 2024
@iLLiCiTiT iLLiCiTiT marked this pull request as draft March 28, 2024 11:12
@iLLiCiTiT
Copy link
Member Author

Waiting for #297 to be merged.

# Conflicts:
#	client/ayon_core/cli.py
#	client/ayon_core/cli_commands.py
#	client/ayon_core/modules/deadline/repository/custom/plugins/GlobalJobPreLoad.py
#	client/ayon_core/modules/royalrender/lib.py
@iLLiCiTiT iLLiCiTiT marked this pull request as ready for review April 2, 2024 14:06
@iLLiCiTiT iLLiCiTiT assigned kalisp and unassigned antirotor Apr 2, 2024
@kalisp kalisp assigned antirotor and unassigned kalisp Apr 3, 2024
@kalisp
Copy link
Member

kalisp commented Apr 3, 2024

Looks good to me, but not sure if this command is actually still being used. @antirotor any idea what this one was for?

@iLLiCiTiT
Copy link
Member Author

Looks good to me, but not sure if this command is actually still being used. @antirotor any idea what this one was for?

Was used in royal render https://github.com/ynput/ayon-core/blob/develop/client/ayon_core/modules/royalrender/rr_root/plugins/control_job/perjob/m50__openpype_publish_render.py.

@iLLiCiTiT iLLiCiTiT merged commit d150da6 into develop Apr 3, 2024
6 of 7 checks passed
@iLLiCiTiT iLLiCiTiT deleted the enhancement/AY-3996_Use-folder-argument branch April 3, 2024 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backwards incompatible type: enhancement Improvement of existing functionality or minor addition
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

5 participants