-
Notifications
You must be signed in to change notification settings - Fork 167
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 upload path for artifacts #505
Conversation
@jezdez sorry for the extra noise but this should be ready now (and hopefully for good). |
There seems to be some overlap with @chrisburr's #503, too |
Is there? I am only touching the workflow file here. It should result in no conflicts. |
Unfortunately I will be touching them as I've found my PR is failing due to the CI testing the latest release rather than the current branch. ( Though it's no trouble for me to rebase on top of this once it's merged. |
Hm, yes, I am seeing this as well now @chrisburr |
Looks like it's working more reliably now, thanks for the tip! |
One more fix for #498 , since apparently the artifacts action doesn't support
../xxx
relative paths.Checks