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

MAYA-114707 test passing array to Maya commands #1831

Merged
merged 6 commits into from
Nov 15, 2021

Conversation

pierrebai-adsk
Copy link
Collaborator

Test passing array of Maya objects, UFE objects or a mix of both to Python select command and Mel select command.

Verified that the test fail in Maya 2022 and work with the new fix in Maya 2023.

Copy link
Collaborator

@ppt-adsk ppt-adsk left a comment

Choose a reason for hiding this comment

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

Thanks for the test! Minor tweaks you can look at.

- Copyright year.
- Enabling pure maya object test on piror version of Maya.
- Fixing comment when disabling tests.
ppt-adsk
ppt-adsk previously approved these changes Nov 12, 2021
ppt-adsk
ppt-adsk previously approved these changes Nov 12, 2021
…taht version.

We don't actually really need to test it anyway, as long as we test with 2022 and 2023 that should be enough. These version details are hard to get right...
@pierrebai-adsk pierrebai-adsk added ready-for-merge Development process is finished, PR is ready for merge and removed ready-for-merge Development process is finished, PR is ready for merge labels Nov 15, 2021
@pierrebai-adsk pierrebai-adsk added the ready-for-merge Development process is finished, PR is ready for merge label Nov 15, 2021
@kxl-adsk kxl-adsk merged commit 15010e4 into dev Nov 15, 2021
@kxl-adsk kxl-adsk deleted the t_bailp/MAYA-114707/ufe-array-in-cmd-tests branch November 15, 2021 19:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-merge Development process is finished, PR is ready for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants