Skip to content

Store entire build directory as CI artifacts#15310

Merged
acdlite merged 3 commits intofacebook:masterfrom
acdlite:fb-bundles-ci
Apr 4, 2019
Merged

Store entire build directory as CI artifacts#15310
acdlite merged 3 commits intofacebook:masterfrom
acdlite:fb-bundles-ci

Conversation

@acdlite
Copy link
Copy Markdown
Collaborator

@acdlite acdlite commented Apr 3, 2019

Updates the Circle CI config to store Facebook bundles as build artifacts. We already do this for our npm packages.

Update: Might as well store entire build/ directory

Updates the Circle CI config to store Facebook bundles as build
artifacts. We already do this for our npm packages.
@acdlite acdlite changed the title Store FB bundles as CI artifacts Store entire build directory as CI artifacts Apr 3, 2019
Copy link
Copy Markdown
Contributor

@bvaughn bvaughn left a comment

Choose a reason for hiding this comment

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

Might as well store entire build/ directory

I think we should use a similar approach as pack_and_store_artifact.sh that just packs up the FB facebook-www into a TGZ so that we have a single thing to download and extract during our sync, rather than each of these individual ones

Screen Shot 2019-04-03 at 1 11 25 PM

So it's easy to download
@acdlite
Copy link
Copy Markdown
Collaborator Author

acdlite commented Apr 3, 2019

Good call

@acdlite
Copy link
Copy Markdown
Collaborator Author

acdlite commented Apr 3, 2019

@bvaughn Ok it now stores the build/ directory as a tarball. The node_modules tarball used by the release script is unchanged.
Screen Shot 2019-04-03 at 4 37 40 PM

Copy link
Copy Markdown
Contributor

@bvaughn bvaughn left a comment

Choose a reason for hiding this comment

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

I dig it.

@acdlite acdlite merged commit 1dcab97 into facebook:master Apr 4, 2019
@facebook-github-bot facebook-github-bot added the React Core Team Opened by a member of the React Core Team label Apr 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed React Core Team Opened by a member of the React Core Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants