Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ability to group multiple features for the powerset #80

Closed
jhpratt opened this issue Oct 24, 2020 · 0 comments · Fixed by #82
Closed

Ability to group multiple features for the powerset #80

jhpratt opened this issue Oct 24, 2020 · 0 comments · Fixed by #82
Assignees
Labels
A-features Area: features (--feature-powerset, --each-feature, etc.) C-enhancement Category: A new feature or an improvement for an existing one

Comments

@jhpratt
Copy link
Contributor

jhpratt commented Oct 24, 2020

With an increasing number of features, the powerset inherently grows exponentially. As such, even the ability to group two features (dependencies in my case) together would provide a significant improvement in CI time. The use-case behind this is for features that are wholly unrelated and do not interact in any way. In my case, I have serde and rand implementations, but enabling one would never cause the other to fail. Because of this, having an option like --group-features serde,rand would be great. The flag could be passed multiple times to create multiple groups.

@taiki-e taiki-e added A-features Area: features (--feature-powerset, --each-feature, etc.) C-enhancement Category: A new feature or an improvement for an existing one labels Oct 24, 2020
@taiki-e taiki-e self-assigned this Oct 24, 2020
@bors bors bot closed this as completed in 96a1c36 Oct 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-features Area: features (--feature-powerset, --each-feature, etc.) C-enhancement Category: A new feature or an improvement for an existing one
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants