Skip to content

Switched to depend on 0.17.0.dev1 and ported tests to pytest#8

Merged
reinecke merged 6 commits intoOpenTimelineIO:mainfrom
reinecke:switch_to_0_17_dev1_and_pytest_migration
Jun 13, 2024
Merged

Switched to depend on 0.17.0.dev1 and ported tests to pytest#8
reinecke merged 6 commits intoOpenTimelineIO:mainfrom
reinecke:switch_to_0_17_dev1_and_pytest_migration

Conversation

@reinecke
Copy link
Copy Markdown
Member

@reinecke reinecke commented Nov 5, 2023

  • Bumped versions in dependencies and ci to 0.17.0.dev1 (rather than extract_adapters branch)
  • Addressed issue where tests weren't explicit about pulling the adapter from source
  • Added src directory to pytest's pathing
  • refactored tests to be pytest-style

This PR supersedes #7.

…t native

Signed-off-by: Eric Reinecke <reinecke.eric@gmail.com>
…v1 in CI rather than extract_adapters

Signed-off-by: Eric Reinecke <reinecke.eric@gmail.com>
Comment thread tests/conftest.py Outdated
Comment thread tests/test_cmx_3600_adapter.py Outdated
Comment thread .github/workflows/ci.yaml Outdated
Comment thread .github/workflows/ci.yaml Outdated
@reinecke
Copy link
Copy Markdown
Member Author

reinecke commented Nov 9, 2023

@JeanChristopheMorinPerso said he may be able to take a run at this :)

…he adapter

Signed-off-by: Jean-Christophe Morin <jean_christophe_morin@hotmail.com>
@JeanChristopheMorinPerso
Copy link
Copy Markdown
Member

It took me a month to get to this, but I finally managed to find some time to do what we talked about.

Signed-off-by: Jean-Christophe Morin <jean_christophe_morin@hotmail.com>
Signed-off-by: Jean-Christophe Morin <jean_christophe_morin@hotmail.com>
@JeanChristopheMorinPerso JeanChristopheMorinPerso force-pushed the switch_to_0_17_dev1_and_pytest_migration branch from 3a40c0e to 6884db0 Compare December 9, 2023 19:30
…itly depend on dev1

Signed-off-by: Eric Reinecke <ereinecke@netflix.com>
@reinecke reinecke merged commit 6644cc7 into OpenTimelineIO:main Jun 13, 2024
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