Skip to content

ci: pin clippy to working version#11401

Merged
fgimenez merged 1 commit intoparadigmxyz:mainfrom
nkysg:clippy_happy
Oct 2, 2024
Merged

ci: pin clippy to working version#11401
fgimenez merged 1 commit intoparadigmxyz:mainfrom
nkysg:clippy_happy

Conversation

@nkysg
Copy link
Contributor

@nkysg nkysg commented Oct 2, 2024

@nkysg nkysg changed the title make clippy happy ci: pin clippy to working version Oct 2, 2024
@fgimenez fgimenez added this pull request to the merge queue Oct 2, 2024
@fgimenez fgimenez added the C-debt A clean up/refactor of existing code label Oct 2, 2024
@fgimenez fgimenez mentioned this pull request Oct 2, 2024
Merged via the queue into paradigmxyz:main with commit 2d59212 Oct 2, 2024
@shekhirin
Copy link
Member

It was failing but didn't seem to be a false positive no?

@nkysg nkysg deleted the clippy_happy branch October 2, 2024 08:31
Thegaram pushed a commit to scroll-tech/reth that referenced this pull request Oct 2, 2024
Thegaram added a commit to scroll-tech/reth that referenced this pull request Oct 2, 2024
* fix: adjust ci workflows

* ci: pin clippy to working version (paradigmxyz#11237)

* ci: pin clippy to working version (paradigmxyz#11401)

* fix integration tests

---------

Co-authored-by: Federico Gimenez <fgimenez@users.noreply.github.com>
Co-authored-by: nk_ysg <nk_ysg@163.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-debt A clean up/refactor of existing code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants