Skip to content

Conversation

@riavalon
Copy link
Contributor

No description provided.

@googlebot googlebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Feb 14, 2017

# create directory and clone test repo
rm -rf /tmp/
rm -rf ./tmp/
Copy link
Member

Choose a reason for hiding this comment

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

Keep /tmp and just call rm -rf $repoPath.

docsPath="/dist/docs"
repoPath="/tmp/material2-docs-content"
docsPath="./dist/docs"
repoPath="./tmp/material2-docs-content"
Copy link
Member

Choose a reason for hiding this comment

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

This should be still /tmp. Just be sure to don't remove the whole /tmp dir later.

@riavalon riavalon force-pushed the fix-docs-content-pushing-script branch from c4328c1 to 30c6e34 Compare February 14, 2017 17:46
Copy link
Member

@devversion devversion left a comment

Choose a reason for hiding this comment

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

LGTM

@devversion devversion added pr: lgtm action: merge The PR is ready for merge by the caretaker labels Feb 14, 2017
@andrewseguin andrewseguin merged commit 60aa9e9 into angular:master Feb 16, 2017
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker cla: yes PR author has agreed to Google's Contributor License Agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants