Skip to content

Fix CI#969

Merged
mergify[bot] merged 3 commits intomasterfrom
fix-ci-again
Mar 2, 2026
Merged

Fix CI#969
mergify[bot] merged 3 commits intomasterfrom
fix-ci-again

Conversation

@psychon
Copy link
Owner

@psychon psychon commented Mar 2, 2026

Fix new unused result warning

The nightly build started failing with the following message:

error: unused result of type `((), ())`
  --> xtrace-example/src/main.rs:91:5
   |
91 |     futures_util::try_join!(future1, future2)?;
   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

and

Pin last working unicode-ident version in msrv-check

See https://github.com/dtolnay/unicode-ident/commit/93f6598c484e61d6c4101179a9a9b52699a7e54f

and

Pin last working syn version in msrv-check

See https://github.com/dtolnay/syn/commit/92a0aa4096e5938025eb643312f2394547cae3f4

A first version if this failed with the following, so the exact version to
update is specified:

error: There are multiple `syn` packages in your project, and the specification `syn` is ambiguous.
Please re-run this command with `-p <spec>` where `<spec>` is one of the following:
  syn@1.0.109
  syn@2.0.117

psychon added 3 commits March 2, 2026 17:58
The nightly build started failing with the following message:

error: unused result of type `((), ())`
  --> xtrace-example/src/main.rs:91:5
   |
91 |     futures_util::try_join!(future1, future2)?;
   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

Signed-off-by: Uli Schlachter <psychon@znc.in>
See dtolnay/syn@92a0aa4

A first version if this failed with the following, so the exact version to
update is specified:

error: There are multiple `syn` packages in your project, and the specification `syn` is ambiguous.
Please re-run this command with `-p <spec>` where `<spec>` is one of the following:
  syn@1.0.109
  syn@2.0.117

Signed-off-by: Uli Schlachter <psychon@znc.in>
@mergify mergify bot merged commit f4c3b99 into master Mar 2, 2026
21 checks passed
@mergify mergify bot deleted the fix-ci-again branch March 2, 2026 17:25
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.

2 participants