Skip to content

Conversation

jmertic
Copy link
Contributor

@jmertic jmertic commented Jun 12, 2020

Looking to move away from terms that could be viewed as racially charged - hopefully this suggestion is ok.

Signed-off-by: John Mertic [email protected]

@codecov-commenter
Copy link

codecov-commenter commented Jun 12, 2020

Codecov Report

Merging #733 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #733   +/-   ##
=======================================
  Coverage   81.88%   81.88%           
=======================================
  Files          72       72           
  Lines        2755     2755           
=======================================
  Hits         2256     2256           
  Misses        499      499           
Flag Coverage Δ
#py27 81.86% <ø> (ø)
#py36 81.86% <ø> (ø)
#py37 81.86% <ø> (ø)

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 894a1e6...f129e90. Read the comment docs.

Copy link
Collaborator

@ssteinbach ssteinbach left a comment

Choose a reason for hiding this comment

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

Good fix!

@jminor
Copy link
Collaborator

jminor commented Jun 12, 2020

This is a good change. @jmertic I don't think we have a CLA on file from you yet.

@jmertic
Copy link
Contributor Author

jmertic commented Jun 12, 2020

@jminor Thanks! How should I go about doing a CLA?

@jminor
Copy link
Collaborator

jminor commented Jun 12, 2020

@jmertic start here: https://opentimelineio.readthedocs.io/en/latest/tutorials/contributing.html
I'm not sure if you want to sign as an individual or as a company - that's up to you.

@jminor
Copy link
Collaborator

jminor commented Jun 25, 2020

CLA received & filed.

@jminor jminor merged commit 1468491 into AcademySoftwareFoundation:master Jun 25, 2020
@jminor
Copy link
Collaborator

jminor commented Jun 25, 2020

Thanks for the contribution!

@ssteinbach ssteinbach added this to the Public Beta 13 milestone Jul 7, 2020
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.

4 participants