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 Windows Release publishing. #476

Merged
merged 1 commit into from
Sep 12, 2023

Conversation

michaelonken
Copy link
Member

Use powershell multi-line command separators (backticks) for long command line call.

Use powershell multi-line command separators (backticks) for long
command line call.
@fedorov
Copy link
Member

fedorov commented Sep 12, 2023

Oh wow! I did have those backticks, but I removed them, since I thought they are an artifact of Appveyor syntax. Thank you for catching this! Not sure why it is failing now.... We also should look (at some point) how to disable package generation and upload when the workflow is triggered on PR.

@michaelonken
Copy link
Member Author

I suspect that windows build above fails since the secret.GA_TOKEN is probably not accessible from my pull request (see also same error on Linux build for this PR).

I tried on my own repo with my own token and it seems to be fine in general.

@fedorov fedorov merged commit b205637 into QIICR:master Sep 12, 2023
0 of 3 checks passed
@fedorov
Copy link
Member

fedorov commented Sep 12, 2023

I am merging this one, since overall PRs do not need to be generating packages, so we can ignore those errors for now I guess.

@michaelonken michaelonken deleted the fix_windows_release_publish branch November 1, 2023 07:58
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