unexpected cfg condition value: serde in declare_raft_types!

[sebadob]

This is not really a bug (I guess), but clippy is complaining about a line in the openraft::declare_raft_types! macro.
I am usually checking pipelines with cargo clippy -- -D warnings which is why I ran into the issue.

It's most probably about the line

#[cfg_attr(feature = "serde", derive(serde::Deserialize, serde::Serialize))]

in the macro definition:

warning: unexpected `cfg` condition value: `serde`
  --> hiqlite/src/store/state_machine/sqlite/mod.rs:11:1
   |
11 | / openraft::declare_raft_types!(
12 | |     pub TypeConfigSqlite:
13 | |         D = QueryWrite,
14 | |         R = Response,
15 | |         Node = Node,
16 | |         SnapshotData = tokio::fs::File,
17 | | );
   | |_^
   |
   = note: expected values for `feature` are: `auto-heal`, `backup`, `cache`, `dashboard`, `default`, `dlock`, `full`, `listen_notify`, `listen_notify_local`, `s3`, `server`, `shutdown-handle`, `sqlite`, and `webpki-roots`
   = help: consider adding `serde` as a feature in `Cargo.toml`
   = note: see <https://doc.rust-lang.org/nightly/rustc/check-cfg/cargo-specifics.html> for more information about checking conditional configuration
   = note: `#[warn(unexpected_cfgs)]` on by default
   = note: this warning originates in the macro `openraft::declare_raft_types` (in Nightly builds, run with -Z macro-backtrace for more info)

When clippy runs, it complains about the serde feature which does only exist for openraft, but not in any other project (most probably) like mine in this case. So I guess this line does not have the desired effect, or is this actually a clippy bug and it does what it should, even when the importing project doesn't have a serde feature?

I am currently using openraft-0.9.17, maybe this is even already gone for 0.10, don't know. I can allow this clippy warning and kind of get rid of the error, but I am not sure if it actually does what it should do in this scenario.

[github-actions]

👋 Thanks for opening this issue!

Get help or engage by:

• /help : to print help messages.
• /assignme : to assign this issue to you.

[SteveLauC]

This is a known issue, and it has been fixed in #1268

[sebadob]

This is a #1270 issue, and it has been fixed in #1268

Thanks.