-
Notifications
You must be signed in to change notification settings - Fork 5
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 small typos in ci README #15
Conversation
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
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
@@ -195,7 +195,7 @@ jobs: | |||
uses: actions/checkout@v4 | |||
- 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! |
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! |
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
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-bitcoin
CI to my BIP-322 crate and found some small things I needed to change to make it work for me.