Skip to content

Take default-features into account#30

Merged
ggwpez merged 6 commits into
masterfrom
dev
Aug 28, 2023
Merged

Take default-features into account#30
ggwpez merged 6 commits into
masterfrom
dev

Conversation

@ggwpez

@ggwpez ggwpez commented Aug 28, 2023

Copy link
Copy Markdown
Owner

Needed for paritytech/polkadot-sdk#1194

Changes to lint propagate-features:

  • If a crate A depends on a crate B and enables its default features, then now all features of B that are transitively enabled by its default feature will also count as being enabled. Note that it only considers features that B enables on itself. It could also check for transitive dependencies in the form of B/G -> C/F -> B/F but it will only allow paths that consist solely of B/* to keep things simple.

ggwpez added 6 commits August 28, 2023 12:34
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
@ggwpez ggwpez merged commit 22e04c5 into master Aug 28, 2023
@ggwpez ggwpez deleted the dev branch August 28, 2023 15:03
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.

1 participant