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 formula class generation when using dots #3117

Merged
merged 2 commits into from May 20, 2022
Merged

fix: brew formula class generation when using dots #3117

merged 2 commits into from May 20, 2022

Conversation

cpheps
Copy link
Contributor

@cpheps cpheps commented May 20, 2022

Resolves: #3116

Change homebrew class generation to match what homebrew expects based on Homebrew's code for generating class names.

…hat homebrew expects

Signed-off-by: Corbin Phelps <corbin.phelps@bluemedora.com>
@pull-request-size pull-request-size bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label May 20, 2022
@caarlos0 caarlos0 changed the title fix: Modified brew formula class generation to match homebrew expectations fix: brew formula class generation when using dots May 20, 2022
@caarlos0 caarlos0 enabled auto-merge (squash) May 20, 2022 13:13
@caarlos0 caarlos0 added the bug Something isn't working label May 20, 2022
@caarlos0 caarlos0 self-assigned this May 20, 2022
@caarlos0 caarlos0 added this to the v1.9.0 milestone May 20, 2022
@vercel vercel bot temporarily deployed to Preview May 20, 2022 13:15 Inactive
@codecov
Copy link

codecov bot commented May 20, 2022

Codecov Report

Merging #3117 (5f22a60) into main (cc27fba) will decrease coverage by 0.24%.
The diff coverage is 100.00%.

❗ Current head 5f22a60 differs from pull request most recent head 4dc9b72. Consider uploading reports for the commit 4dc9b72 to get more accurate results

@@            Coverage Diff             @@
##             main    #3117      +/-   ##
==========================================
- Coverage   84.66%   84.41%   -0.25%     
==========================================
  Files         113      113              
  Lines        9273     9300      +27     
==========================================
  Hits         7851     7851              
- Misses       1149     1176      +27     
  Partials      273      273              
Impacted Files Coverage Δ
internal/pipe/brew/brew.go 84.36% <100.00%> (ø)
internal/client/github.go 34.45% <0.00%> (-3.46%) ⬇️

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 cc27fba...4dc9b72. Read the comment docs.

@caarlos0 caarlos0 merged commit 8dde97c into goreleaser:main May 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Brew Formula Class Name not Correct with Version Added
2 participants