Skip to content

Commit 40c5d96

Browse files
committed
Merge rust-bitcoin#4605: Remove _all_ the trailing whitespace
6cb4e29 Remove _all_ the trailing whitespace (Tobin C. Harding) Pull request description: Recently we tried to make the rustfmt bot remove trailing whitespace but wrote the `sed` command incorrectly - it only removes a single character. Tested by running the new command on rust-bitcoin#4600. ACKs for top commit: jamillambert: ACK 6cb4e29 apoelstra: ACK 6cb4e29; successfully ran local tests Tree-SHA512: dd5c4ad3b38848ff5603e5b7077a9b3b62e6e3b232583ac62a486ba3ee71e27d3fe0d404c53110c484d44ca98dee9013be5ff232f4705e9c2ed6be04705395d0
2 parents c6c690a + 6cb4e29 commit 40c5d96

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

.github/workflows/cron-weekly-rustfmt.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ jobs:
1414
components: rustfmt
1515
- name: Run Nightly rustfmt
1616
# Run the formatter and manually remove trailing whitespace.
17-
run: cargo +nightly fmt && find . -type f -name '*.rs' -exec sed -i 's/ $//' {} \;
17+
run: cargo +nightly fmt && git ls-files -- '*.rs' -z | xargs sed -E -i'' -e 's/[[:space:]]+$//'
1818
- name: Get the current date
1919
run: echo "date=$(date +'%Y-%m-%d')" >> $GITHUB_ENV
2020
- name: Create Pull Request

0 commit comments

Comments
 (0)