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
chore: standardize release script #270
Conversation
I've noticed we rarely update the version on this project, so I'm assuming we're publishing with the same version. Is that intentional, or should I be adding a version update into the process? |
@jye-sf I think we just haven't released in a while because there was nothing to change. When we do a release, we should bump the version. |
@@ -27,7 +27,9 @@ | |||
"lint": "eslint src/ tests/", | |||
"format": "prettier --write '**/*.{js,json,md,html,css}'", | |||
"format:check": "prettier --check '**/*.{js,json,md,html,css}'", | |||
"release": "npm publish --access public", | |||
"release": "yarn release:publish", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Full service command will require a script that can read args (for version upgrade) and proper permissions to merge into master. Keeping this script as-is until we have that functionality.
package.json
Outdated
"release": "yarn release:publish", | ||
"release:version": "npm version", | ||
"release:publish": "npm publish --access public --registry=https://registry.npmjs.org", | ||
"release:publish:ci": "npm publish --access public --registry=https://registry.npmjs.org --dry-run", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will remove --dry-run
after review for safety.
executors: | ||
node: | ||
docker: | ||
- image: circleci/node:14 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- image: circleci/node:14 | |
- image: cimg/node:14.19.1 |
.circleci/config.yml
Outdated
keys: | ||
- v1-yarn-deps-{{ checksum "yarn.lock" }} | ||
- v1-yarn-deps- |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
keys: | |
- v1-yarn-deps-{{ checksum "yarn.lock" }} | |
- v1-yarn-deps- | |
key: v1-yarn-deps-{{ checksum "yarn.lock" }} |
Standardize the naming for our release scripts:
yarn release -> full service release
yarn release:version -> update the version
yarn release:publish -> publish the project