Skip to content
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

CI: Fix macos native build #1412

Merged
merged 6 commits into from
Feb 13, 2025

Apply acc06bc91f80ddf4e015dcdf0b984bbdbfcb5ca3 fix also to pip upgrade

49f151f
Select commit
Loading
Failed to load commit list.
Merged

CI: Fix macos native build #1412

Apply acc06bc91f80ddf4e015dcdf0b984bbdbfcb5ca3 fix also to pip upgrade
49f151f
Select commit
Loading
Failed to load commit list.
Cirrus CI / lint [bionic] failed Feb 13, 2025 in 3m 27s

Task Summary

Instruction lint failed in 02:14

Details

✅ 00:52 clone
✅ 00:18 merge_base
❌ 02:14 lint

In contrib/merge-prs.sh line 136:
    PR_ID=$(echo $line | cut -d ' ' -f 6 | tr -d :)
                 ^---^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean: 
    PR_ID=$(echo "$line" | cut -d ' ' -f 6 | tr -d :)


In contrib/merge-prs.sh line 137:
    PR_ID_ALT=$(echo $line | cut -d ' ' -f 8 | tr -d :)
                     ^---^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean: 
    PR_ID_ALT=$(echo "$line" | cut -d ' ' -f 8 | tr -d :)


In contrib/merge-prs.sh line 153:
        git -C "$WORKTREE" merge $HASH --no-ff -m "Merge $HASH into merged_master ($CHAIN PR $PR_ID)"
                                 ^---^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean: 
        git -C "$WORKTREE" merge "$HASH" --no-ff -m "Merge $HASH into merged_master ($CHAIN PR $PR_ID)"

For more information:
  https://www.shellcheck.net/wiki/SC2034 -- BTC_COMMITS appears unused. Verif...
  https://www.shellcheck.net/wiki/SC2086 -- Double quote to prevent globbing ...
^---- failure generated from test/lint/lint-shell.sh
src/qt/bitcoinunits.cpp:66: Lsat ==> Last, slat, sat
src/qt/bitcoinunits.cpp:79: Lsat ==> Last, slat, sat
src/simplicity/jets-secp256k1.c:717: signatue ==> signature
src/simplicity/jets.c:594: upto ==> up to
src/simplicity/jets.c:628: upto ==> up to
src/simplicity/jets.c:1070: satifying ==> satisfying
src/simplicity/jets.c:1088: precondtion ==> precondition
src/simplicity/jets.c:1119: guarentees ==> guarantees
src/simplicity/jets.c:1134: exectued ==> executed
src/simplicity/jets.c:1176: remainer ==> remainder
src/simplicity/jets.c:1190: intermidiate ==> intermediate
src/simplicity/jets.c:1191: remainer ==> remainder
src/simplicity/primitive/elements/env.c:263: explict ==> explicit
src/simplicity/primitive/elements/env.c:267: remaning ==> remaining
src/simplicity/primitive/elements/env.c:500: initalize ==> initialize
src/simplicity/primitive/elements/env.c:531: entires ==> entries
src/simplicity/primitive/elements/env.c:531: initalized ==> initialized
src/simplicity/primitive/elements/env.c:532: compiliers ==> compilers
src/simplicity/rsort.c:14: represenation ==> representation
src/simplicity/rsort.c:17: archtectures ==> architectures
src/simplicity/rsort.c:141: excecpt ==> except
src/simplicity/rsort.c:172: poping ==> popping, pooping
src/simplicity/secp256k1/README.md:52: uninitalized ==> uninitialized
src/simplicity/secp256k1/util.h:99: suported ==> supported
src/simplicity/sha256.c:126: implemenation ==> implementation
src/simplicity/sha256.h:133: implemenation ==> implementation
src/simplicity/type.h:88: arguements ==> arguments
src/simplicity/type.h:93: arguements ==> arguments
^ Warning: codespell identified likely spelling errors. Any false positives? Add them to the list of ignored words in test/lint/lint-spelling.ignore-words.txt