OpenTimelineIO / otio-aaf-adapter

OpenTimelineIO Advanced Authoring Format (AAF) Adapter
Apache License 2.0
15 stars 6 forks source link

OpenTimelineIO AAF Adapter Removal Tasks #1

Open markreidvfx opened 1 year ago

markreidvfx commented 1 year ago

The following tasks need to be resolved before AAF Adapter can safely be removed from OpenTimelineIO. Please let me know if there are any other blocking issues to add to this task list.

The pull request to remove the AAF Adapter from OTIO is here https://github.com/AcademySoftwareFoundation/OpenTimelineIO/pull/1348

Blocking Tasks

Possible Blocking Tasks

JeanChristopheMorinPerso commented 1 year ago

Regarding CLA and DCO, none of the repositories in the OpenTimelineIO organization have them enabled. I personally think it's fine, but obviously I'm not a member of the TSC so that's just my personal opinion.

CodeCov and Dependabot are really easy to enable. See https://docs.codecov.com/docs/github-2-getting-a-codecov-account-and-uploading-coverage and https://docs.github.com/en/code-security/dependabot/dependabot-version-updates/about-dependabot-version-updates (I think you might have to actually enable Dependabot in the repo settings).

I can help with some of these and the packaging/build side of things too.

markreidvfx commented 1 year ago

@jminor said he was going to look into the CLA stuff.
Thanks @JeanChristopheMorinPerso , I have DCO and CodeCov working in my fork.

The annoying thing with CodeCov at the moment is pytest is running against a installed wheel and CodeCov doesn't like the file paths if you do that. Trying to think of the best approach to deal with this. pip install -e . works fine, but I like the idea of running the test against the wheels like you helped me do in my repos. pytest-cov doesn't seem have many options help do that.

JeanChristopheMorinPerso commented 1 year ago

Maybe https://docs.codecov.com/docs/fixing-paths?

markreidvfx commented 1 year ago

That looks like it might work, I'll give it a try!