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

OpenTimelineIO AAF Adapter Removal Tasks #1

Open
11 of 18 tasks
markreidvfx opened this issue Aug 4, 2022 · 5 comments
Open
11 of 18 tasks

OpenTimelineIO AAF Adapter Removal Tasks #1

markreidvfx opened this issue Aug 4, 2022 · 5 comments

Comments

@markreidvfx
Copy link
Member

markreidvfx commented Aug 4, 2022

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
AcademySoftwareFoundation/OpenTimelineIO#1348

Blocking Tasks

Possible Blocking Tasks

  • Migrate OTIO AAF Adapter Issues or link
  • Migrate OTIO AAF Adapter Pull Requests?
  • What OTIO version do we test against? multiple?
  • Any additional docs need to move/remove from OTIO?
  • Improve front page README
@JeanChristopheMorinPerso
Copy link
Member

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
Copy link
Member Author

@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
Copy link
Member

@markreidvfx
Copy link
Member Author

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

@timlehr
Copy link
Collaborator

timlehr commented Sep 21, 2024

I think this can be closed, no? @markreidvfx

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

No branches or pull requests

3 participants