Skip to content

[rush] Add 'variants' feature to Rush - #722

Merged
Pete Gonzalez (octogonz) merged 12 commits into
microsoft:masterfrom
ThomasMichon:variants
Oct 25, 2018
Merged

[rush] Add 'variants' feature to Rush#722
Pete Gonzalez (octogonz) merged 12 commits into
microsoft:masterfrom
ThomasMichon:variants

Conversation

@ThomasMichon

Copy link
Copy Markdown
Member

This change implements the proposal in issue #721.

Comment thread apps/rush-lib/src/api/RushConfiguration.ts Outdated

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I added this to BaseRushAction because a lot of actions needed to touch the Shrinkwrap file. I could try scoping it to exactly the set of actions that need it, I suppose.

@octogonz Pete Gonzalez (octogonz) changed the title Add 'variants' feature to Rush [rush] Add 'variants' feature to Rush Aug 10, 2018
@ThomasMichon

Copy link
Copy Markdown
Member Author

I added a configuration option to opt-in to variants, allowing a list of names and descriptions. I also scoped the acceptance of the parameter to only some commands, not all.

Comment thread apps/rush-lib/src/cli/test/CommandLineHelp.test.ts Outdated
Comment thread apps/rush-lib/src/api/Variants.ts Outdated
Comment thread apps/rush-lib/src/logic/PackageChangeAnalyzer.ts Outdated
Comment thread apps/rush-lib/src/logic/PackageChangeAnalyzer.ts Outdated
Comment thread apps/rush-lib/src/schemas/rush.schema.json Outdated
Comment thread apps/rush-lib/src/api/RushConfiguration.ts Outdated

@octogonz Pete Gonzalez (octogonz) left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

:shipit:

@octogonz
Pete Gonzalez (octogonz) merged commit 904e698 into microsoft:master Oct 25, 2018
@octogonz

Copy link
Copy Markdown
Collaborator

This was published as Rush 5.4.0.

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.

3 participants