-
Notifications
You must be signed in to change notification settings - Fork 8
Fix small typos in ci README #15
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
Conversation
apoelstra
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.
utACK 39a0b85
apoelstra
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.
utACK 39a0b85
|
Interesting. I definitely only ran the |
ci/README.md
Outdated
| - name: Read nightly version | ||
| id: read_toolchain | ||
| run: echo "nightly_version=$(cat nightly-version)" >> $GITHUB_OUTPUT | ||
| run: echo "nightly_version=$(cat nightly_version)" >> $GITHUB_OUTPUT |
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.
This is wrong, the file is called nightly-version.
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.
Ok then line 181 must be wrong. I'll change it.
rust-bitcoin-maintainer-tools/ci/README.md
Line 181 in 4f17a05
| $ cat nightly_version |
|
Thanks man, can you squash into a single commit please. |
For sure! |
tcharding
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.
ACK b44c53b
|
Thanks man! |
apoelstra
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.
utACK b44c53b
apoelstra
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.
utACK b44c53b
I was going through the README trying to add the
rust-bitcoinCI to my BIP-322 crate and found some small things I needed to change to make it work for me.