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

Limit release of official branch #344

Merged
merged 1 commit into from
Aug 2, 2021
Merged

Conversation

clamoriniere
Copy link
Collaborator

@clamoriniere clamoriniere commented Jul 30, 2021

What does this PR do?

Only run the CI release workflow on specific branches: main and v* such as v0.6.x

Motivation

Do not create random release when adding a "custom" tag.

Additional Notes

Anything else we should know when reviewing?

Describe your test plan

Write there any instructions and details you may have to test your PR.

@clamoriniere clamoriniere requested a review from a team as a code owner July 30, 2021 09:49
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This pull request does not contain a valid label. Please add one of the following labels: bug, enhancement, documentation

@clamoriniere clamoriniere added this to the v0.7 milestone Jul 30, 2021
@clamoriniere clamoriniere added the bug Something isn't working label Jul 30, 2021
@codecov-commenter
Copy link

codecov-commenter commented Jul 30, 2021

Codecov Report

Merging #344 (987e044) into main (ddbd7b5) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #344   +/-   ##
=======================================
  Coverage   63.98%   63.98%           
=======================================
  Files          61       61           
  Lines        6694     6694           
=======================================
  Hits         4283     4283           
  Misses       2104     2104           
  Partials      307      307           
Flag Coverage Δ
unittests 63.98% <ø> (ø)

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 ddbd7b5...987e044. Read the comment docs.

@clamoriniere clamoriniere merged commit 7f05429 into main Aug 2, 2021
@clamoriniere clamoriniere deleted the clamoriniere/fix-release branch August 2, 2021 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants