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

Fix failing cargo-udeps #5203

Merged
merged 3 commits into from
Feb 6, 2024
Merged

Fix failing cargo-udeps #5203

merged 3 commits into from
Feb 6, 2024

Conversation

jimmygchen
Copy link
Member

@jimmygchen jimmygchen commented Feb 6, 2024

Issue Addressed

CI is currently blocked as the curve25519-dalek nightly build has started failing since rustc 1.78.0-nightly (f067fd608 2024-02-05) :

https://github.com/sigp/lighthouse/actions/runs/7793900607/job/21254345866

I've submitted a PR to fix this: dalek-cryptography/curve25519-dalek#619

This PR patches the curve25519-dalek and should be reverted once they release a fix for this.

@jimmygchen jimmygchen added ready-for-review The code is ready for review infra-ci labels Feb 6, 2024
Cargo.toml Outdated Show resolved Hide resolved
@michaelsproul michaelsproul 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 Feb 6, 2024
@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. labels Feb 6, 2024
@michaelsproul
Copy link
Member

Lets open an issue to remove the patch before 5.0 (if possible) too

@jimmygchen
Copy link
Member Author

Thanks, raised #5204

@michaelsproul
Copy link
Member

@Mergifyio queue

Copy link

mergify bot commented Feb 6, 2024

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at 5cc29e4

mergify bot added a commit that referenced this pull request Feb 6, 2024
@mergify mergify bot merged commit 5cc29e4 into sigp:unstable Feb 6, 2024
28 of 29 checks passed
danielrachi1 pushed a commit to danielrachi1/lighthouse that referenced this pull request Feb 14, 2024
* Attempt to fix nightly build.

* Update Cargo.toml to pin a commit

* Update Cargo.lock
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.

2 participants