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: decouple project_name guessing from the release pipe #4335

Merged
merged 2 commits into from Oct 14, 2023

Conversation

caarlos0
Copy link
Member

the release's defaults run before the project's does, so, usually the github/gitlab/gitea names are set.

however, in some cases, the release's defaults might be skipped, in which case they'll be empty.

this breaks things like goreleaser changelog, especially on non-go repositories.

this pr tries to extract the project name from the git remote url in the project's defaulter.

it might be possible now to move it to run before the release defaulter, even.

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
@caarlos0 caarlos0 added the bug Something isn't working label Sep 29, 2023
@caarlos0 caarlos0 self-assigned this Sep 29, 2023
@pull-request-size pull-request-size bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Sep 29, 2023
@codecov
Copy link

codecov bot commented Sep 29, 2023

Codecov Report

Merging #4335 (b1b661e) into main (b7218b0) will decrease coverage by 0.01%.
Report is 2 commits behind head on main.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #4335      +/-   ##
==========================================
- Coverage   83.90%   83.89%   -0.01%     
==========================================
  Files         133      133              
  Lines       12716    12711       -5     
==========================================
- Hits        10669    10664       -5     
  Misses       1630     1630              
  Partials      417      417              
Files Coverage Δ
internal/pipe/project/project.go 87.80% <100.00%> (+3.93%) ⬆️

... and 1 file with indirect coverage changes

@cloudflare-pages
Copy link

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: b1b661e
Status: ✅  Deploy successful!
Preview URL: https://58f0813f.goreleaser.pages.dev
Branch Preview URL: https://project-name-guess.goreleaser.pages.dev

View logs

Copy link

@harshsingh32 harshsingh32 left a comment

Choose a reason for hiding this comment

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

lgtm

@caarlos0 caarlos0 merged commit 3cfefcc into main Oct 14, 2023
10 of 12 checks passed
@caarlos0 caarlos0 deleted the project-name-guess branch October 14, 2023 21:59
@github-actions github-actions bot added this to the v1.22.0 milestone Oct 14, 2023
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/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants