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

Reworded error message #1482

Merged
merged 1 commit into from Nov 18, 2022
Merged

Conversation

jminor
Copy link
Collaborator

@jminor jminor commented Nov 18, 2022

This PR changes an error message which was flagged here: https://security.lfx.linuxfoundation.org/#/a092M00001If9uZQAR/non-inclusive-language

Signed-off-by: Joshua Minor <joshm@pixar.com>
@codecov-commenter
Copy link

Codecov Report

Merging #1482 (3ac3da9) into main (b6fbca2) will increase coverage by 0.08%.
The diff coverage is 0.00%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1482      +/-   ##
==========================================
+ Coverage   78.89%   78.97%   +0.08%     
==========================================
  Files         200      200              
  Lines       21724    21779      +55     
  Branches     4403     4406       +3     
==========================================
+ Hits        17139    17200      +61     
+ Misses       2355     2350       -5     
+ Partials     2230     2229       -1     
Flag Coverage Δ
py-unittests 78.97% <0.00%> (+0.08%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...entimelineio-bindings/otio_serializableObjects.cpp 26.59% <0.00%> (ø)
tests/test_cmx_3600_adapter.py 99.59% <0.00%> (+0.02%) ⬆️
src/opentimelineio/serialization.cpp 47.99% <0.00%> (+0.18%) ⬆️
...opentimelineio/opentimelineio/adapters/cmx_3600.py 85.06% <0.00%> (+0.28%) ⬆️
tests/test_plugin_detection.py 91.46% <0.00%> (+0.67%) ⬆️
...-opentimelineio/opentimelineio/plugins/manifest.py 91.50% <0.00%> (+4.36%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b6fbca2...3ac3da9. Read the comment docs.

@meshula meshula merged commit e86d91e into AcademySoftwareFoundation:main Nov 18, 2022
@jminor jminor deleted the reword branch November 19, 2022 05:15
MichaelPlug pushed a commit to MichaelPlug/OpenTimelineIO that referenced this pull request Aug 5, 2023
Signed-off-by: Joshua Minor <joshm@pixar.com>

Signed-off-by: Joshua Minor <joshm@pixar.com>
Signed-off-by: Michele Spina <michelespina96@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants