FYI, updating rust edition makes it hard to use dora node api with older rust version. I would maybe wait to upgrade rust edition unless it is strictly necessary

I think the Rust 2024 edition was introduced in Rust 1.85, which is the lowest Rust version we support. So it should be ok.

I force-pushed the branch to remove commits 8867ea2 and 64af27b from this PR. These are commits of another PR that are part of main already, no need to re-apply them.

Maybe we can consider using Squash and merge when merging PRs in the future, this will make the history of main branch much cleaner.

Depends on the PR. For example, I prefer multiple commits for larger PRs that can be split into multiple separate steps. This way, the commits on the main branch are smaller and git blame output becomes more useful.

For this PR, keeping the the formatting and clippy changes in separate commits makes the history and diffs easier to read in my opinion.