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(brew): Remove deprecated bottle :unneeded #2591

Merged
merged 2 commits into from
Oct 19, 2021
Merged

fix(brew): Remove deprecated bottle :unneeded #2591

merged 2 commits into from
Oct 19, 2021

Conversation

mcornick
Copy link
Contributor

Remove bottle :unneeded from brew formula generated by goreleaser.

bottle :unneeded now provokes a warning from Homebrew:
Warning: Calling bottle :unneeded is deprecated! There is no replacement.

Homebrew apparently deprecated bottle :unneeded in Homebrew/brew@f65d525

@pull-request-size pull-request-size bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Oct 19, 2021
Fixes this warning from Homebrew:
`Warning: Calling bottle :unneeded is deprecated! There is no replacement.`
@mcornick mcornick changed the title Remove deprecated bottle :unneeded from brews fix(brew): Remove deprecated bottle :unneeded Oct 19, 2021
Copy link
Member

@caarlos0 caarlos0 left a comment

Choose a reason for hiding this comment

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

Thanks!

@caarlos0 caarlos0 added the bug Something isn't working label Oct 19, 2021
@caarlos0 caarlos0 added this to In progress in Board via automation Oct 19, 2021
@caarlos0 caarlos0 self-assigned this Oct 19, 2021
@caarlos0 caarlos0 enabled auto-merge (squash) October 19, 2021 18:20
@mcornick
Copy link
Contributor Author

I'm chasing down that extra newline that was added - will update the PR with a fix.

auto-merge was automatically disabled October 19, 2021 18:47

Head branch was pushed to by a user without write access

@caarlos0 caarlos0 enabled auto-merge (squash) October 19, 2021 18:58
Board automation moved this from In progress to Reviewer approved Oct 19, 2021
auto-merge was automatically disabled October 19, 2021 19:01

Head branch was pushed to by a user without write access

@mcornick
Copy link
Contributor Author

Thanks for your patience - I think I've got the formula template fixed up properly now. (Newlines in go templates always seem to bite me...)

@caarlos0
Copy link
Member

Newlines in go templates always seem to bite me..

you and me both eheh

@codecov
Copy link

codecov bot commented Oct 19, 2021

Codecov Report

Merging #2591 (95e2fcb) into master (a2ff2b8) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2591   +/-   ##
=======================================
  Coverage   82.87%   82.87%           
=======================================
  Files          98       98           
  Lines        7207     7207           
=======================================
  Hits         5973     5973           
  Misses       1043     1043           
  Partials      191      191           

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 a2ff2b8...95e2fcb. Read the comment docs.

@caarlos0 caarlos0 merged commit ba9b75a into goreleaser:master Oct 19, 2021
Board automation moved this from Reviewer approved to Done Oct 19, 2021
@caarlos0
Copy link
Member

thanks!

jabley added a commit to jabley/kubeval that referenced this pull request Oct 20, 2021
Use the next-gen language images as per
https://circleci.com/docs/2.0/circleci-images/

Change proposed to create a new build which will pick up
goreleaser/goreleaser#2591
@mcornick mcornick deleted the bottle-unneeded-deprecated branch October 20, 2021 19:59
jason-fugue pushed a commit to fugue/regula that referenced this pull request Nov 9, 2021
`brew install` or `brew upgrade` for `regula` currently shows the following:
```sh
Warning: Calling bottle :unneeded is deprecated! There is no replacement.
Please report this issue to the fugue/regula tap (not Homebrew/brew or Homebrew/core):
  /usr/local/Homebrew/Library/Taps/fugue/homebrew-regula/Formula/regula.rb:9
```

The generation is fixed in goreleaser/goreleaser#2591 and available with GoReleaser v0.183.0.

Signed-off-by: Timofey Ilinykh <ilinytim@gmail.com>
rothgar added a commit to rothgar/awsls that referenced this pull request Dec 16, 2021
I believe this includes a newer version of the homebrew  formula which fixes jckuester#45

via goreleaser/goreleaser#2591
rothgar added a commit to rothgar/kubecolor that referenced this pull request Dec 16, 2021
Updates the homebrew formula to remove bottle.

fixes hidetatz#81 via goreleaser/goreleaser#2591
jckuester pushed a commit to jckuester/awsls that referenced this pull request Feb 9, 2022
I believe this includes a newer version of the homebrew  formula which fixes #45

via goreleaser/goreleaser#2591
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 20, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
No open projects
Board
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants