Skip to content

Conversation

@jotak
Copy link
Member

@jotak jotak commented Apr 6, 2022

Uses the 'ok-to-test' label to trigger the build/push

Uses the 'ok-to-test' label to trigger the build/push
@jotak jotak requested review from eranra and ronensc April 6, 2022 09:26
@codecov-commenter
Copy link

Codecov Report

Merging #173 (ea52f6d) into main (3d0abe3) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #173   +/-   ##
=======================================
  Coverage   58.20%   58.20%           
=======================================
  Files          55       55           
  Lines        3137     3137           
=======================================
  Hits         1826     1826           
  Misses       1190     1190           
  Partials      121      121           
Flag Coverage Δ
unittests 58.20% <ø> (ø)

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 3d0abe3...ea52f6d. Read the comment docs.

Comment on lines +52 to +57
github.rest.issues.createComment({
issue_number: context.issue.number,
owner: context.repo.owner,
repo: context.repo.repo,
body: 'New image: ${{ steps.push-to-quay.outputs.registry-paths }}. It will expire after two weeks.'
})
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'm a bit confused. If this workflow is triggered by a PR, then what is the issue in context where the comment is created?

Copy link
Member Author

Choose a reason for hiding this comment

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

in github API, a PR is apparently some special kind of issue. But the comment is actually created on the PR

@eranra eranra linked an issue Apr 6, 2022 that may be closed by this pull request
Copy link
Collaborator

@eranra eranra left a comment

Choose a reason for hiding this comment

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

@jotak looks good to me

@jotak jotak merged commit 29cb7dd into netobserv:main Apr 6, 2022
@jotak
Copy link
Member Author

jotak commented Apr 6, 2022

thanks

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.

Limit create pre-merge images to specific tag

4 participants