Skip to content

Conversation

@Akagi201
Copy link
Contributor

Previous logic incorrectly pushed both the .exe file name AND the plain name as an additional path component on Windows, resulting in a non-existent path

@stefan-mysten
Copy link
Collaborator

Thanks for the fix. The CI is failing due to formatting issues.
Try running cargo fmt --all -- --check and push the changes.

@Akagi201
Copy link
Contributor Author

Thanks for the fix. The CI is failing due to formatting issues. Try running cargo fmt --all -- --check and push the changes.

Sorry I forgot it, I have formatted the code and force push it.

@stefan-mysten
Copy link
Collaborator

Cherry-picked in #146, thanks @Akagi201!

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