Skip to content

Fix ci issues - #9514

Merged
mergify[bot] merged 3 commits into
sigp:unstablefrom
eserilev:fix-ci-issues
Jun 23, 2026
Merged

Fix ci issues#9514
mergify[bot] merged 3 commits into
sigp:unstablefrom
eserilev:fix-ci-issues

Conversation

@eserilev

@eserilev eserilev commented Jun 22, 2026

Copy link
Copy Markdown
Member

Issue Addressed

  • Remove deprecated geth flag
  • Remove duplicate entries for crate syn

@eserilev eserilev added the ready-for-review The code is ready for review label Jun 22, 2026
@eserilev

Copy link
Copy Markdown
Member Author

ignore flaky ci issue this should be g2g

@mergify

mergify Bot commented Jun 22, 2026

Copy link
Copy Markdown

Some required checks have failed. Could you please take a look @eserilev? 🙏

@mergify mergify Bot added waiting-on-author The reviewer has suggested changes and awaits thier implementation. and removed ready-for-review The code is ready for review labels Jun 22, 2026

@michaelsproul michaelsproul left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

happy to merge as is to just fix CI

Comment thread Cargo.lock Outdated
[[package]]
name = "data-encoding-macro"
version = "0.1.19"
version = "0.1.18"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this the root cause of the syn thing? seems weird to downgrade this

I think just changing syn 1 to 2 also works

this is some weird cargo stuff we haven't got to the bottom of

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The rust-libp2p update here seems to have updated data-encoding-macro which downgraded syn

@mergify

mergify Bot commented Jun 22, 2026

Copy link
Copy Markdown

Queued — the merge queue status continues in this comment ↓.

@mergify

mergify Bot commented Jun 22, 2026

Copy link
Copy Markdown

Merge Queue Status

This pull request spent 29 minutes 30 seconds in the queue, including 27 minutes 59 seconds running CI.

Required conditions to merge
  • #approved-reviews-by >= 1 [🛡 GitHub branch protection]
  • check-success=local-testnet-success
  • check-success=test-suite-success

@mergify mergify Bot added the queued label Jun 22, 2026
@michaelsproul michaelsproul added ready-for-merge This PR is ready to merge. and removed waiting-on-author The reviewer has suggested changes and awaits thier implementation. queued labels Jun 22, 2026
@mergify mergify Bot added the queued label Jun 22, 2026
mergify Bot added a commit that referenced this pull request Jun 22, 2026
mergify Bot added a commit that referenced this pull request Jun 22, 2026
mergify Bot added a commit that referenced this pull request Jun 22, 2026
mergify Bot added a commit that referenced this pull request Jun 22, 2026
@mergify
mergify Bot merged commit 8c4b21c into sigp:unstable Jun 23, 2026
38 checks passed
@mergify mergify Bot removed the queued label Jun 23, 2026
dknopik pushed a commit to dknopik/lighthouse that referenced this pull request Jul 9, 2026
- Remove deprecated geth flag
- Remove duplicate entries for crate `syn`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

infra-ci ready-for-merge This PR is ready to merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants