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.

Sign up for GitHub

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

Jump to bottom

Skip new test if UFE attribute metadata not available #2834

Merged
merged 1 commit into from
Jan 23, 2023
Merged

Skip new test if UFE attribute metadata not available #2834

merged 1 commit into from
Jan 23, 2023

Conversation

dj-mcg
Copy link
Collaborator

@dj-mcg dj-mcg commented Jan 23, 2023

UFE attribute metadata API was introduced in UFE 3. The new testAttributeNiceNames should be disabled if the API is now available (UFE 2 and earlier)

@dj-mcg
Skip new test if UFE attribute metadata not available
f3f3e6f 
UFE attribute metadata API was introduced in UFE 3. The new
testAttributeNiceNames should be disabled if the API is now available
(UFE 2 and earlier)
seando-adsk
seando-adsk approved these changes Jan 23, 2023
View reviewed changes
Copy link
Collaborator

@seando-adsk seando-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 finding and fixing this. It shows we have a missing build/test scenario in our preflight.

@seando-adsk seando-adsk added unit test Related to unit tests (both python or c++) ready-for-merge Development process is finished, PR is ready for merge labels Jan 23, 2023
@seando-adsk seando-adsk merged commit 681ae52 into Autodesk:dev Jan 23, 2023
@dj-mcg dj-mcg deleted the pr/Fix_Test_for_UFE2 branch January 24, 2023 20:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@seando-adsk seando-adsk seando-adsk approved these changes

Assignees

@ecp-maya-devops-adsk ecp-maya-devops-adsk

Labels
ready-for-merge Development process is finished, PR is ready for merge unit test Related to unit tests (both python or c++)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dj-mcg @seando-adsk @ecp-maya-devops-adsk