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

Fix goreleaser #2333

Merged
merged 1 commit into from Jan 6, 2022
Merged

Fix goreleaser #2333

merged 1 commit into from Jan 6, 2022

Conversation

ciarams87
Copy link
Member

Proposed changes

There's a bug in the pipeline when we push a tag - goreleaser won't work with the "--id" flag when we are releasing the binaries. See https://github.com/nginxinc/kubernetes-ingress/runs/4727732721?check_suite_focus=true for the failed run.

Checklist

Before creating a PR, run through this checklist and mark each as complete.

@github-actions github-actions bot added the bug An issue reporting a potential bug label Jan 6, 2022
@codecov-commenter
Copy link

Codecov Report

Merging #2333 (fd5a5f0) into master (1cf4513) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2333   +/-   ##
=======================================
  Coverage   53.68%   53.68%           
=======================================
  Files          48       48           
  Lines       14199    14199           
=======================================
  Hits         7623     7623           
  Misses       6338     6338           
  Partials      238      238           

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 1cf4513...fd5a5f0. Read the comment docs.

@ciarams87 ciarams87 merged commit f3236ff into master Jan 6, 2022
@ciarams87 ciarams87 deleted the bug/fix-goreleaser branch January 6, 2022 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An issue reporting a potential bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants