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.

Sign up for GitHub

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

Jump to bottom

Support splitting by multiple properties in `--rule-list-split` option #351

Merged

bmish merged 1 commit into main from split-by-multi

Dec 19, 2022

Owner

bmish commented Dec 17, 2022 •

edited

Fixes #291.

This is not yet splitting the lists correctly. The same rule should not be included in multiple lists.
Ensure the CSV string behavior is correct.
Ensure this will be compatible with Allow custom function to be specified for splitting/ordering of sub-lists when using --rule-list-split option #294. We might want to just use an underlying function even if the user isn't using one.

bmish added the enhancement label

bmish mentioned this pull request

Better normalize pathRuleList option internally #352

Merged

bmish force-pushed the split-by-multi branch 4 times, most recently from ec54e07 to 8379b23 Compare

December 19, 2022 23:24


          feat: enable using multiple properties in --rule-list-split

43f6fe5

bmish force-pushed the split-by-multi branch from 8379b23 to 43f6fe5 Compare

December 19, 2022 23:28

bmish marked this pull request as ready for review

December 19, 2022 23:29

bmish merged commit f8bd2da into main

bmish deleted the split-by-multi branch

December 19, 2022 23:37

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment