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

Changed to vMajor tag for pre-commit in ci_update_dependencies.yml #82

Merged
merged 2 commits into from
Dec 6, 2022

Conversation

francescalb
Copy link
Contributor

Intead of using a moving V1 tag, we change to using the actual version tag

@francescalb francescalb linked an issue Nov 24, 2022 that may be closed by this pull request
3 tasks
@codecov-commenter
Copy link

codecov-commenter commented Dec 6, 2022

Codecov Report

Merging #82 (bba2fe4) into main (0991d57) will not change coverage.
The diff coverage is n/a.

❗ Current head bba2fe4 differs from pull request most recent head 7155668. Consider uploading reports for the commit 7155668 to get more accurate results

@@           Coverage Diff           @@
##             main      #82   +/-   ##
=======================================
  Coverage   53.69%   53.69%           
=======================================
  Files           3        3           
  Lines         311      311           
=======================================
  Hits          167      167           
  Misses        144      144           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Collaborator

@CasperWA CasperWA left a comment

Choose a reason for hiding this comment

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

All good for my part now :)

francescalb and others added 2 commits December 6, 2022 15:58
Intead of using a moving V1 tag, we change to using the actual version tag

change version in all .md
Update ci_tests workflow and docs.
Remove unused files and suggest changes.
@CasperWA CasperWA force-pushed the 81-drop-using-a-vmajor-dynamic-tag branch from bba2fe4 to 7155668 Compare December 6, 2022 14:59
@CasperWA CasperWA merged commit 7155668 into main Dec 6, 2022
@CasperWA CasperWA deleted the 81-drop-using-a-vmajor-dynamic-tag branch December 6, 2022 15:00
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.

Drop using a vMAJOR dynamic tag
3 participants