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

Removes global package.json, updates circleci to use node cli. #1234

Merged
merged 15 commits into from
Oct 27, 2020

Conversation

smurrayatwork
Copy link
Contributor

No description provided.

@clairesunstudio
Copy link
Contributor

@smurrayatwork this looks good, can you remove the circle test trigger from this branch and update the cron time to Tuesday 10:45am?

Co-authored-by: Minghua Sun <[email protected]>
# Identify the committer i.e. CircleCI deployment bot
- run: git config --global user.email "[email protected]"
- run: git config --global user.name "CircleCI Deployment Bot"
- run: node common/scripts/install-run-rushx.js release-tag
- run: export NODE_PATH=$(npm root --quiet -g) && node ./scripts/release-tag.js
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- run: export NODE_PATH=$(npm root --quiet -g) && node ./scripts/release-tag.js
- run: export NODE_PATH=$(npm root --quiet -g) && node ./scripts/release-tag.js

I guess this is the same as scripts/release-tag.js just want to make sure it works

@clairesunstudio clairesunstudio merged commit 79b99b1 into develop Oct 27, 2020
@clairesunstudio clairesunstudio deleted the develop--without-package.json branch October 27, 2020 14:32
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.

2 participants