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

Release v0.15.0 changes #1403

Merged

Conversation

ssteinbach
Copy link
Collaborator

@ssteinbach ssteinbach commented Sep 20, 2022

Preparing for Release v0.15.0:

  • freeze runners to their current versions according to this: https://github.com/actions/runner-images
  • drop .dev1 suffix
  • check the README.md for any updates
    • update wording to reflect contemporary project better, emphasizing features of C++ vs Python libraries
    • add C++ example snippet
    • add link to "presentations" wiki
    • update VFX Platform supported list
  • update CONTRIBUTORS.md
  • bump version in CMakeLists.txt to 0.15.0 (that was supposed to happen at the beginning of the release but that step was missed)
  • c++ code formatted

@JeanChristopheMorinPerso
Copy link
Member

I'm not too sure why but the CI failed with weird errors.

@ssteinbach
Copy link
Collaborator Author

I saw that too. I'm not sure what is going on, I tried restarting the jobs and they failed again. Maybe he values i got from the table weren't right? Or I had a typo? I'll look later.

@codecov-commenter
Copy link

codecov-commenter commented Sep 22, 2022

Codecov Report

Merging #1403 (69eadec) into main (cbef407) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1403   +/-   ##
=======================================
  Coverage   85.99%   85.99%           
=======================================
  Files         200      200           
  Lines       20934    20934           
  Branches     2459     2459           
=======================================
  Hits        18002    18002           
  Misses       2331     2331           
  Partials      601      601           
Flag Coverage Δ
py-unittests 85.99% <ø> (ø)

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


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 cbef407...69eadec. Read the comment docs.

@ssteinbach
Copy link
Collaborator Author

I'm not too sure why but the CI failed with weird errors.

Got it working. I'm going to merge other release changes into this pr- setting the version # and so on

@ssteinbach ssteinbach changed the title freeze versions ahead of release Release v0.15.0 changes Sep 22, 2022
@ssteinbach
Copy link
Collaborator Author

@meshula I forget, are the VFX platform versions listed in the README.md accurate for this release?

vfx platform 2018-2021

@meshula
Copy link
Collaborator

meshula commented Sep 23, 2022

@ssteinbach vfx platform should be current year + 3, so 2022, 2021, 2020, and 2019.

README.md Outdated Show resolved Hide resolved
@ssteinbach ssteinbach marked this pull request as ready for review September 23, 2022 17:14
@ssteinbach ssteinbach merged commit 3703ac7 into AcademySoftwareFoundation:main Sep 26, 2022
@ssteinbach ssteinbach deleted the main.release.0.15.0 branch September 26, 2022 18:49
MichaelPlug pushed a commit to MichaelPlug/OpenTimelineIO that referenced this pull request Aug 5, 2023
* freeze versions ahead of release
* remove path
* remove .dev1 suffix
* bump version number in CMakeLists.txt to 0.15
* Update README.md
* Update CONTRIBUTORS.md
* update supported VFX platform list
* add link to Presentations wiki page
* tweak some settings in clang format
* Run make format over the c++ code
* match autoformatted C++ in the version map

Signed-off-by: Stephan Steinbach <[email protected]>
Co-authored-by: Nick Porcino <[email protected]>
Signed-off-by: Michele Spina <[email protected]>
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.

6 participants