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

[rush] Initial implementation of Rush build cache feature. #2390

Merged
merged 36 commits into from
Dec 22, 2020

Conversation

iclanton
Copy link
Member

@iclanton iclanton commented Dec 11, 2020

See this issue for design: #2393

This branch will build this repo with this feature: https://github.com/iclanton/rushstack/tree/ianc/build-cache-example

Test Notes

In a clone of this repo without the buildCache experiment turned on:

  • Ran rush rebuild --to . repeatedly in the @rushstack/tree-pattern folder without issue
  • Ran rush build --to . repeatedly in the @rushstack/tree-pattern folder without issue
  • Ran rush rebuild --to . repeatedly in the @rushstack/heft folder without issue
  • Ran rush build --to . repeatedly in the @rushstack/heft folder without issue

@iclanton iclanton force-pushed the ianc/rush-build-cache-az branch 3 times, most recently from 12836e8 to 3d9e7d9 Compare December 19, 2020 03:41
@iclanton iclanton marked this pull request as ready for review December 21, 2020 00:43
@iclanton iclanton force-pushed the ianc/rush-build-cache-az branch 2 times, most recently from eb09819 to a5110c7 Compare December 21, 2020 20:50
@iclanton iclanton force-pushed the ianc/rush-build-cache-az branch from a5110c7 to eb45744 Compare December 21, 2020 22:16
@iclanton iclanton force-pushed the ianc/rush-build-cache-az branch from eb45744 to fb22804 Compare December 21, 2020 22:43
@octogonz
Copy link
Collaborator

Please update the PR description with test notes.

Since this is an "experimental" feature, the main validation of interest would be a proof that this won't break any existing Rush features when the experiment is disabled.

@iclanton iclanton force-pushed the ianc/rush-build-cache-az branch from d9f0d92 to fef1f51 Compare December 22, 2020 01:07
@iclanton iclanton force-pushed the ianc/rush-build-cache-az branch from c5619ef to d03d963 Compare December 22, 2020 02:24
@iclanton iclanton merged commit 9d175a4 into microsoft:master Dec 22, 2020
@iclanton iclanton deleted the ianc/rush-build-cache-az branch December 22, 2020 04:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants