Skip to content
This repository has been archived by the owner on Aug 12, 2024. It is now read-only.

Makefile: various fixes #893

Merged
merged 1 commit into from
May 15, 2024

Conversation

joelanford
Copy link
Member

@joelanford joelanford commented May 15, 2024

  • explicitly configure shell
  • change CONTAINER_RUNTIME error to warning
  • fix broken GORELEASER_ARGS setup

@joelanford joelanford requested a review from a team as a code owner May 15, 2024 19:05
@joelanford
Copy link
Member Author

/hold
Trying to convince myself the changes will actually result in the publishing step happening if we cut a new tag after this merges.

@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label May 15, 2024
- explicitly configure shell
- change CONTAINER_RUNTIME error to warning
- fix broken GORELEASER_ARGS setup
@joelanford joelanford changed the title Makefile: explicitly configure shell and change CONTAINER_RUNTIME error to warning Makefile: various fixes May 15, 2024
@joelanford
Copy link
Member Author

I think I found the issue with the 1.21 release GHA. I messed up the GORELEASER_ARGS variable setup in a previous PR.

I think this is likely to fix the issue
/hold cancel

@openshift-ci openshift-ci bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label May 15, 2024
Copy link

codecov bot commented May 15, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 37.95%. Comparing base (bd62a21) to head (6a27013).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #893   +/-   ##
=======================================
  Coverage   37.95%   37.95%           
=======================================
  Files           9        9           
  Lines         843      843           
=======================================
  Hits          320      320           
  Misses        481      481           
  Partials       42       42           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@rashmigottipati rashmigottipati left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label May 15, 2024
@grokspawn grokspawn added this pull request to the merge queue May 15, 2024
Merged via the queue into operator-framework:main with commit 19e1839 May 15, 2024
9 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants