Skip to content

Conversation

@findepi
Copy link
Member

@findepi findepi commented Nov 28, 2024

Temporarily pin toolchain version until problems reported by newer clippy release are solved.

Temporarily pin toolchain version until problems reported by newer
clippy release are solved.
@findepi
Copy link
Member Author

findepi commented Nov 28, 2024

cc @Dandandan @alamb let's maybe merge this in, since the PR builds already started to fail -- #13595 (comment)

@Dandandan Dandandan merged commit 2b37018 into apache:main Nov 28, 2024
26 checks passed
@findepi findepi deleted the findepi/temporarily-pin-toolchain-version-to-avoid-clippy-4acc89 branch November 28, 2024 20:39
@findepi
Copy link
Member Author

findepi commented Nov 28, 2024

thanks for the merge!

findepi added a commit to findepi/datafusion that referenced this pull request Nov 28, 2024
…)"

This reverts commit 2b37018.
Toolchain pinning is no longer needed.
jonahgao pushed a commit that referenced this pull request Nov 29, 2024
* Apply clippy fixes

`dev/rust_lint.sh` no longer passes for me, maybe because of `rustup
update`. This is first portion of fixes suggested by clippy.

* Fix typo

Co-authored-by: Oleks V <comphead@users.noreply.github.com>

* Suppress missing docs clippy check in test code

* Revert "Temporarily pin toolchain version to avoid clippy (#13598)"

This reverts commit 2b37018.
Toolchain pinning is no longer needed.

---------

Co-authored-by: Daniël Heres <danielheres@gmail.com>
Co-authored-by: Oleks V <comphead@users.noreply.github.com>
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