-
Notifications
You must be signed in to change notification settings - Fork 990
GitHub ci/cd fix #3815
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
GitHub ci/cd fix #3815
Conversation
| with: | ||
| files: | | ||
| target/release/grin-${{ github.ref_name }}-win-x86_64.zip | ||
| target/release/grin-${{ github.ref_name }}-win-x86_64-sha256sum.txt |
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.
i don't see what changed on this line.
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 correct line ending existed at this file, so git provided LF ending, as I edited file at Github website. We can create.gitattributes file with * text=auto content, it will treat all files as text files and convert to OS's line ending on checkout and back to LF on commit automatically.
| - name: Checkout | ||
| uses: actions/checkout@v3 | ||
| - name: Tests | ||
| run: cargo test --release --all |
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.
or on this line.
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.
some Github nextline formatting I guess, CR LF
tromp
left a comment
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.
Looks fine, except for non-visible changes, which should be avoided.
GitHub ci/cd fix (mimblewimble#3815)
windows-2019towindows-2022.builtdependency to avoid link issue at old git2 lib: Windows build does not link with Advapi32.lib rust-lang/git2-rs#1142.