Skip to content

ci(goreleaser): Fix the failure of release workflow#1263

Merged
ehsandeep merged 3 commits intoprojectdiscovery:mainfrom
suzuki-shunsuke:ci-fix-release
Oct 30, 2024
Merged

ci(goreleaser): Fix the failure of release workflow#1263
ehsandeep merged 3 commits intoprojectdiscovery:mainfrom
suzuki-shunsuke:ci-fix-release

Conversation

@suzuki-shunsuke
Copy link
Copy Markdown
Contributor

@suzuki-shunsuke suzuki-shunsuke commented Oct 30, 2024

--rm-dist was deprecated.

https://goreleaser.com/deprecations/?h=dep#-rm-dist

The release workflow failed.

https://github.com/projectdiscovery/naabu/actions/runs/11593776126/job/32278527815

/opt/hostedtoolcache/goreleaser-action/2.3.2/x64/goreleaser release -f .goreleaser/linux.yml --rm-dist
  ⨯ command failed                                   error=unknown flag: --rm-dist
Error: The process '/opt/hostedtoolcache/goreleaser-action/2.3.2/x64/goreleaser' failed with exit code 1

--rm-dist was deprecated.

https://goreleaser.com/deprecations/?h=dep#-rm-dist

The release workflow failed.

https://github.com/projectdiscovery/naabu/actions/runs/11593776126/job/32278527815

```
/opt/hostedtoolcache/goreleaser-action/2.3.2/x64/goreleaser release -f .goreleaser/linux.yml --rm-dist
  ⨯ command failed                                   error=unknown flag: --rm-dist
Error: The process '/opt/hostedtoolcache/goreleaser-action/2.3.2/x64/goreleaser' failed with exit code 1
```
Fix the warning.

```console
$ goreleaser check v2/.goreleaser/linux.yml
  • only configurations files on  version: 2  are supported, yours is  version: 0 , please update your configuration
  • checking                                 path=v2/.goreleaser/linux.yml
  • 1 configuration file(s) validated
  • thanks for using goreleaser!
```
@suzuki-shunsuke suzuki-shunsuke marked this pull request as ready for review October 30, 2024 14:09
@suzuki-shunsuke
Copy link
Copy Markdown
Contributor Author

suzuki-shunsuke commented Oct 30, 2024

Oh? Assets were released somehow.
Did anyone release them manually?

https://github.com/projectdiscovery/naabu/releases/tag/v2.3.2

image

@ehsandeep
Copy link
Copy Markdown
Member

@suzuki-shunsuke we fixed it in #1262 and yes, since automed CI failed, we pushed the binary based on latest commit on main.

@suzuki-shunsuke
Copy link
Copy Markdown
Contributor Author

Oh, I see. #1262 was merged in dev branch, so I missed it.

@suzuki-shunsuke
Copy link
Copy Markdown
Contributor Author

Should I close this pr?

@ehsandeep ehsandeep merged commit 5e3bf29 into projectdiscovery:main Oct 30, 2024
@suzuki-shunsuke suzuki-shunsuke deleted the ci-fix-release branch October 30, 2024 14:15
@suzuki-shunsuke
Copy link
Copy Markdown
Contributor Author

Thank you for merging this pr!

@suzuki-shunsuke
Copy link
Copy Markdown
Contributor Author

BTW, I can't find any document about dev branch.

https://github.com/search?q=repo%3Aprojectdiscovery%2Fnaabu%20dev&type=code

Should we send pull requests to dev branch?
I'm not sure the development flow of this project.

@ehsandeep
Copy link
Copy Markdown
Member

@suzuki-shunsuke That’s correct. For most OSS projects, we push new code to the dev branch and move it to main when creating a release. You can always pull the latest changes from dev and create a PR against the dev branch.

@suzuki-shunsuke
Copy link
Copy Markdown
Contributor Author

Got it. Thank you for your explanation!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants