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: Grep for exact error message #639

Closed
wants to merge 1 commit into from

Conversation

tcharding
Copy link
Member

@tcharding tcharding commented Aug 8, 2023

cargo +nightly output of panic recently changed breaking our grep statement by adding the code line number and a newline.

Grep for the exact second line of the error message. Note we use dots instead of version numbers to reduce the maintenance burden.

`cargo +nightly` output of panic recently changed breaking our grep
statement by adding the code line and a newline.

Grep for the exact second line of the error message.
@tcharding
Copy link
Member Author

tcharding commented Aug 8, 2023

I think CI will pass once this PR is on top of #640

This was referenced Aug 8, 2023
@tcharding
Copy link
Member Author

Combined with #640 into #641.

@tcharding tcharding closed this Aug 8, 2023
apoelstra added a commit that referenced this pull request Aug 10, 2023
92778ef CI: Pin cc in ASAN no_std_test crate (Tobin C. Harding)
5e6dd8a CI: Use bash instead of sh (Tobin C. Harding)
cf5f103 Target panic message at lib users (Tobin C. Harding)
ec9c964 Allow stuff after unconditional panic (Tobin C. Harding)
3bbf083 no_std_test: Remove internal_features (Tobin C. Harding)
cff7a3d CI: Grep for exact error message (Tobin C. Harding)
79e184f CI: Update MSRV pins (Tobin C. Harding)

Pull request description:

  CI has a bunch of things broken.

  This is #640 followed by #639 followed by  a few addition fixes to get CI green again. Includes clippy warnings in feature gated code which is not strictly necessary and a fix to a panic message, also not strictly necessary.

ACKs for top commit:
  apoelstra:
    ACK 92778ef

Tree-SHA512: f99b01e17fade7df394299bdb6bf385bec3f88d6568d43962238049b33a94c364d48c266acb358e72a48dd55a4aac6300ace6478b0821275b89cb86eba639d8b
@tcharding tcharding deleted the 08-08-fix-ci-grep branch October 9, 2023 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant