Skip to content

Replace derivative dependency with derive-where#7324

Merged
gui1117 merged 5 commits intoparitytech:masterfrom
conr2d:derive-where
Jan 30, 2025
Merged

Replace derivative dependency with derive-where#7324
gui1117 merged 5 commits intoparitytech:masterfrom
conr2d:derive-where

Conversation

@conr2d
Copy link
Copy Markdown
Contributor

@conr2d conr2d commented Jan 24, 2025

Description

Close #7122.

This PR replaces the unmaintained derivative dependency with derive-where.

Integration

This PR doesn't change the public interfaces.

Review Notes

The derivative crate, previously used to derive basic traits for structs with generics or enums, is no longer actively maintained. It has been replaced with the derive-where crate, which offers a more straightforward syntax while providing the same features as derivative.

@conr2d conr2d requested a review from a team as a code owner January 24, 2025 13:28
@bkchr bkchr added the R0-no-crate-publish-required The change does not require any crates to be re-published. label Jan 25, 2025
Copy link
Copy Markdown
Contributor

@gui1117 gui1117 left a comment

Choose a reason for hiding this comment

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

Syntax is nice.

@gui1117 gui1117 enabled auto-merge January 28, 2025 09:10
@gui1117 gui1117 disabled auto-merge January 30, 2025 08:51
@gui1117 gui1117 enabled auto-merge January 30, 2025 08:52
@gui1117 gui1117 added this pull request to the merge queue Jan 30, 2025
Merged via the queue into paritytech:master with commit 0d644ca Jan 30, 2025
ordian added a commit that referenced this pull request Feb 3, 2025
* master:
  Remove warnings by cleaning up the `Cargo.toml` (#7416)
  [Backport] Version bumps from stable2412-1 + prdocs reorg (#7401)
  fix pre-dispatch PoV underweight for ParasInherent (#7378)
  malus-collator: implement malicious collator submitting same collation to all backing groups (#6924)
  `fatxpool`: use tracing for logging (#6897)
  Improvements for Weekly bench (#7390)
  Replace derivative dependency with derive-where (#7324)
  Add support for feature `pallet_balances/insecure_zero_ed` in benchmarks and testing (#7379)
  Fix Snowbridge benchmark tests (#7296)
  Bridges small nits/improvements (#7383)
  Migrating cumulus-pallet-session-benchmarking to Benchmarking V2  (#6564)
  [pallet-revive] implement the block author API  (#7198)
  Use checked math in frame-balances named_reserve (#7365)
  move installation of frame-omni-bencher into a cmd.py itself (#7372)
  remove old bench & revert the frame-weight-template (#7362)
  ci: fix workflow permissions (#7366)
  [net/libp2p] Use raw `Identify` observed addresses to discover external addresses (#7338)
  Improve `set_validation_data` error message. (#7359)
  Implement pallet view function queries (#4722)
Ank4n pushed a commit that referenced this pull request Feb 6, 2025
# Description

Close #7122.

This PR replaces the unmaintained `derivative` dependency with
`derive-where`.

## Integration

This PR doesn't change the public interfaces.

## Review Notes

The `derivative` crate, previously used to derive basic traits for
structs with generics or enums, is no longer actively maintained. It has
been replaced with the `derive-where` crate, which offers a more
straightforward syntax while providing the same features as
`derivative`.

---------

Co-authored-by: Guillaume Thiolliere <gui.thiolliere@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

R0-no-crate-publish-required The change does not require any crates to be re-published.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replace usage of derivative

4 participants