-
Notifications
You must be signed in to change notification settings - Fork 446
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 error publishing docs #1619
Conversation
Well, it's a different error at least
|
ed62849
to
19c0bee
Compare
Successfully published! 🎉 https://www.scala-sbt.org/sbt-native-packager/ I used b6c9142 to test publishing from this PR. I'll drop the commit before merging. Importantly, I had to open this PR from the main repo and not my fork in order for Github Actions to have write permission. You can see the permissions granted by I made the following changes outside of this PR:
|
19c0bee
to
891c6c0
Compare
publish: | ||
runs-on: ubuntu-20.04 | ||
steps: | ||
- uses: actions/checkout@v2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a reason to use this old version?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No reason, I just copied it from release.yml
Amazing work 🙏💪thanks a lot |
Fixes #1597
Supersedes #1618
I'm opening this PR from this repository instead of my fork. Hopefully that will fix the permissions issue.