Skip to content
This repository was archived by the owner on Dec 18, 2024. It is now read-only.

Publish scripts #76

Merged
merged 3 commits into from
Jan 4, 2017
Merged

Publish scripts #76

merged 3 commits into from
Jan 4, 2017

Conversation

ericjim
Copy link
Member

@ericjim ericjim commented Jan 4, 2017

It is assumed that the deployer has access to both instances (dev and prod)

@@ -1,6 +1 @@
{}
{
Copy link
Member Author

@ericjim ericjim Jan 4, 2017

Choose a reason for hiding this comment

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

The command firebase deploy defaults to material2-docs-dev, however, we can control this by adding the instance name as a parameter (refer to publish-dev in package.json).

@ericjim ericjim requested a review from jelbourn January 4, 2017 21:24
Some members of the team ran into issues using yarn. At the time of
writing, it is believed 0.16.0 works.
@jelbourn
Copy link
Member

jelbourn commented Jan 4, 2017

LGTM

We'll eventually want to have this command also pull down the appropriate static assets

@jelbourn jelbourn merged commit 0ab6cf6 into angular:master Jan 4, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants