Skip to content

Conversation

@benalleng
Copy link
Collaborator

This adds a commit onto some existing commits that we know fail ci to see if an update to cargo mutants fixes the problem

Pull Request Checklist

Please confirm the following before requesting review:

@benalleng
Copy link
Collaborator Author

RIP

@benalleng benalleng closed this Dec 3, 2025
@benalleng benalleng reopened this Dec 3, 2025
@coveralls
Copy link
Collaborator

coveralls commented Dec 3, 2025

Pull Request Test Coverage Report for Build 19909489050

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 83.539%

Totals Coverage Status
Change from base Build 19824041153: 0.0%
Covered Lines: 8988
Relevant Lines: 10759

💛 - Coveralls

@benalleng benalleng force-pushed the cargo-ci-failure-test branch from b62270b to 37e8bd3 Compare December 3, 2025 21:12
@benalleng
Copy link
Collaborator Author

@spacebear21 could you sanity check this one? We just want to filter the diff for .rs files anyways right?

@benalleng benalleng requested a review from spacebear21 December 3, 2025 21:16
@spacebear21
Copy link
Collaborator

We just want to filter the diff for .rs files anyways right?

That makes sense to me. I broke the dart workflow with a bad push to #1208, but if you rebase this PR with just your fix i'll merge it and rebase mine on top of it.

@benalleng
Copy link
Collaborator Author

I can just drop your commits to keep the history cleaner

@benalleng benalleng force-pushed the cargo-ci-failure-test branch from 37e8bd3 to e50b132 Compare December 3, 2025 21:30
@benalleng benalleng changed the title [Test] Cargo ci failure test Cargo mutants filter diff only for rust files Dec 3, 2025
@benalleng benalleng marked this pull request as ready for review December 3, 2025 21:31
@spacebear21 spacebear21 merged commit 640a20f into payjoin:master Dec 3, 2025
10 checks passed
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.

3 participants