-
Notifications
You must be signed in to change notification settings - Fork 69
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
Is setup-gpg still needed? #194
Comments
If that can help, running this script instead of the |
You can remove setup-gpg. It’s no longer needed with the latest sbt-ci-release. |
Should I send in a pr to remove it from this repo then? Because the instructions include a step that copies the release file from this repo which still includes it. https://github.com/olafurpg/sbt-ci-release#github-actions |
Yeah, that would be great. I had assumed setup-gpg didn’t cause issues with releases so I didn’t make an active effort to deprecate it yet. I’ll follow up and archive the repo with appropriate docs. |
This seems to be sometimes causing problems with the newest version of sbt-ci-release. Following up on the conversation in sbt#194 I'm removing this since there is a step in the docs that copies this release.yml.
Closed via #195 (comment) |
As mentioned in sbt/sbt-ci-release#194. Let's see.
I ran into a hiccup that took me a little while to figure out today. I kept hitting on the followinng error message when trying to do a release:
There also seems to be similar errors reported in a few other issues:
- #173 (comment)
- #172
- #170
While some of them had different solutions one of them hinted at removing sbt-gpg, which caused me to try that, which finally worked for me and allowed for my release to work: https://github.com/scoverage/scalac-scoverage-plugin/actions/runs/802328069
I see that there was a pr to remove setup-pgp here, but I also see there is a commit later where you added it back in here.
I'm a bit confused as I have some projects that seem to require it, and then others that don't. Do you have any info on when this may or may not be required? I'll gladly expand the README if there is more info about this, but I'm a bit unsure if it's actually needed and when it may be needed.
The text was updated successfully, but these errors were encountered: