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: semantic-release setup #18

Merged
merged 1 commit into from
Sep 12, 2021
Merged

fix: semantic-release setup #18

merged 1 commit into from
Sep 12, 2021

Conversation

gr2m
Copy link
Collaborator

@gr2m gr2m commented Sep 12, 2021

I think this should fix the semantic-release setup. I had some problems when I was testing it with my fork. I use pretty much the same setup in many of my actions, I'm confused why this caused errors in the first place. Maybe because I ran it on a fork?

@gr2m
Copy link
Collaborator Author

gr2m commented Sep 12, 2021

I'm just going to merge it in to see if that fixes the release setup.

If it doesn't, I'll debug this in a separate repository until I figure out what's up :)

@gr2m gr2m merged commit 44d26c5 into main Sep 12, 2021
@gr2m gr2m deleted the 17/semantic-release-fix branch September 12, 2021 20:59
@github-actions
Copy link

🎉 This PR is included in version 1.0.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@gr2m
Copy link
Collaborator Author

gr2m commented Sep 12, 2021

@stefanbuck I've no idea what this error means 😞

image

I'll check in with GitHub support. For the time being I'll fix the v1 branch manually so that we can continue to use the action :D

@stefanbuck
Copy link
Owner

I think it is caused by this line which ends with a double quotation mark, but there is no counter part.

git push --force https://x-access-token:${GITHUB_TOKEN}@github.com/${GITHUB_REPOSITORY}.git HEAD:refs/heads/v1"

@gr2m
Copy link
Collaborator Author

gr2m commented Sep 12, 2021

That was it 👍🏼 Great catch!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants