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

Update tests to handle issue with skin mesh #1206

Merged
merged 1 commit into from
Oct 13, 2023
Merged

Conversation

rafacanton
Copy link
Contributor

@rafacanton rafacanton commented Oct 13, 2023

Update a test that specifically checks the values of a PropertyField returned by the skin operator, that needs update:

  • This PR comments the conflicted part.
  • Then the PR in TFS will be completed.
  • Then dpf_standalone will be updated.
  • Then the commented part will be uncommented.

@codecov
Copy link

codecov bot commented Oct 13, 2023

Codecov Report

Merging #1206 (56819ca) into master (8605173) will increase coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1206      +/-   ##
==========================================
+ Coverage   87.25%   87.30%   +0.05%     
==========================================
  Files          81       81              
  Lines        9224     9224              
==========================================
+ Hits         8048     8053       +5     
+ Misses       1176     1171       -5     

@rafacanton rafacanton added the bug Something isn't working label Oct 13, 2023
@rafacanton rafacanton self-assigned this Oct 13, 2023
@rafacanton rafacanton merged commit 5ee39a2 into master Oct 13, 2023
31 of 32 checks passed
@rafacanton rafacanton deleted the rcanton/bug888224 branch October 13, 2023 08:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants