Skip to content

Commit

Permalink
chore: Fix npm/node to v6 and latest LTS (#441)
Browse files Browse the repository at this point in the history
  • Loading branch information
frankieyan authored Feb 22, 2021
1 parent 6dc59f9 commit 3c008ba
Show file tree
Hide file tree
Showing 24 changed files with 37 additions and 20 deletions.
2 changes: 1 addition & 1 deletion .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ Feel free to leave unchecked or remove the lines that are not applicable.
- [ ] Updated docs (storybooks, readme)
- [ ] Executed `npm run validate` and made sure no errors / warnings were shown
- [ ] Described changes in `CHANGELOG.md`
- [ ] Bumped version in `package.json`
- [ ] Bumped version in `package.json` and `package-lock.json` (`npm --no-git-tag-version version <major|minor|patch>`) [ref](https://docs.npmjs.com/cli/v6/commands/npm-version)
- [ ] Updated all static build artifacts (`npm run build-all`)

## Versioning
Expand Down
2 changes: 2 additions & 0 deletions .github/codeql/codeql-config.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
paths:
- src
3 changes: 2 additions & 1 deletion .github/workflows/codeql-analysis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,8 +30,9 @@ jobs:
# Initializes the CodeQL tools for scanning.
- name: Initialize CodeQL
uses: github/codeql-action/init@v1
with:
config-file: ./.github/codeql/codeql-config.yml
# Override language selection by uncommenting this and choosing your languages
# with:
# languages: go, javascript, csharp, python, cpp, java

# Autobuild attempts to build any compiled languages (C/C++, C#, or Java).
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/coverage-reporting.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ jobs:
- uses: actions/checkout@v2
- uses: actions/setup-node@v1
with:
node-version: 12
node-version: 14
- name: Install dependencies
run: npm set progress=false && CYPRESS_INSTALL_BINARY=0 HUSKY_SKIP_INSTALL=1 npm ci --silent
- name: Run tests and collect coverage
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ jobs:
- uses: actions/checkout@v1
- uses: actions/setup-node@v1
with:
node-version: 12
node-version: 14
- run: npm ci
- run: npm run lint
- run: npm run type-check
Expand All @@ -24,7 +24,7 @@ jobs:
# Publish to GitHub package registry
- uses: actions/setup-node@v1
with:
node-version: 12
node-version: 14
registry-url: https://npm.pkg.github.com/
scope: '@doist'
- run: npm publish
Expand All @@ -34,7 +34,7 @@ jobs:
# Publish to npm registry
- uses: actions/setup-node@v1
with:
node-version: 12
node-version: 14
registry-url: https://registry.npmjs.org/
scope: '@doist'
- run: npm publish
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/pull_request.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ jobs:
- uses: actions/checkout@v1
- uses: actions/setup-node@v1
with:
node-version: 12
node-version: 14
- run: npm ci
- run: npm run lint
- run: npm run type-check
Expand Down
1 change: 1 addition & 0 deletions .npmrc
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
engine-strict=true
1 change: 1 addition & 0 deletions .nvmrc
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
v14.15.5
5 changes: 5 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,11 @@

Reactist follows [semantic versioning](https://semver.org/) and doesn't introduce breaking changes (API-wise) in minor or patch releases. However, the appearance of a component might change in a minor or patch release so keep an eye on redesigns and make sure your app still looks and feels like you expect it.

## 9.0.0

- [Build] The project now requires node v14.5.5+ and npm v6.14.11+ to install and run.
- We're skipping right over v8.0.0 as its tag has already been created previously

## 7.2.1

- [Tweak] Adds `defaultValue` and ensures additional props are carried forward on the `Select` component.
Expand Down
8 changes: 7 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -44,11 +44,17 @@ You can find our changelog [here](./CHANGELOG.md).

# Development

First clone the repository, and run its setup task:
We leverage `package.json`'s `engines` field to specify the node/npm versions to be used while in development. The easiest way to switch quickly is through [fnm](https://github.com/Schniz/fnm).

It's worth noting that fnm can automatically switch to a project's specified node version when it encounters a `.nvmrc` file. To do so, use the `--use-on-cd` flag when [generating your shell's config script with `fnm env`](https://github.com/Schniz/fnm#fnm-env).

Once fnm is installed, clone the repository. Then, switch to the required node version, and run its setup task:

```sh
git clone https://github.com/doist/reactist.git
cd reactist
fnm install
fnm use
npm run setup
```

Expand Down
2 changes: 1 addition & 1 deletion docs/iframe.html
Original file line number Diff line number Diff line change
Expand Up @@ -71,4 +71,4 @@
}</script><style>#root[hidden],
#docs-root[hidden] {
display: none !important;
}</style></head><body><div class="sb-nopreview sb-wrapper"><div class="sb-nopreview_main"><h1 class="sb-nopreview_heading sb-heading">No Preview</h1><p>Sorry, but you either have no stories or none are selected somehow.</p><ul><li>Please check the Storybook config.</li><li>Try reloading the page.</li></ul><p>If the problem persists, check the browser console, or the terminal you've run Storybook from.</p></div></div><div class="sb-errordisplay sb-wrapper"><pre id="error-message" class="sb-heading"></pre><pre class="sb-errordisplay_code"><code id="error-stack"></code></pre></div><div id="root"></div><div id="docs-root"></div><script src="runtime~main.e8da7b7e63150a6ee1ad.bundle.js"></script><script src="vendors~main.e8da7b7e63150a6ee1ad.bundle.js"></script><script src="main.e8da7b7e63150a6ee1ad.bundle.js"></script></body></html>
}</style></head><body><div class="sb-nopreview sb-wrapper"><div class="sb-nopreview_main"><h1 class="sb-nopreview_heading sb-heading">No Preview</h1><p>Sorry, but you either have no stories or none are selected somehow.</p><ul><li>Please check the Storybook config.</li><li>Try reloading the page.</li></ul><p>If the problem persists, check the browser console, or the terminal you've run Storybook from.</p></div></div><div class="sb-errordisplay sb-wrapper"><pre id="error-message" class="sb-heading"></pre><pre class="sb-errordisplay_code"><code id="error-stack"></code></pre></div><div id="root"></div><div id="docs-root"></div><script src="runtime~main.929b84d083b192d4babb.bundle.js"></script><script src="vendors~main.929b84d083b192d4babb.bundle.js"></script><script src="main.929b84d083b192d4babb.bundle.js"></script></body></html>

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions docs/main.929b84d083b192d4babb.bundle.js.map

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 0 additions & 1 deletion docs/main.e8da7b7e63150a6ee1ad.bundle.js.map

This file was deleted.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions docs/runtime~main.929b84d083b192d4babb.bundle.js.map

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 0 additions & 1 deletion docs/runtime~main.e8da7b7e63150a6ee1ad.bundle.js.map

This file was deleted.

3 changes: 3 additions & 0 deletions docs/vendors~main.929b84d083b192d4babb.bundle.js

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions docs/vendors~main.929b84d083b192d4babb.bundle.js.map

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 0 additions & 3 deletions docs/vendors~main.e8da7b7e63150a6ee1ad.bundle.js

This file was deleted.

1 change: 0 additions & 1 deletion docs/vendors~main.e8da7b7e63150a6ee1ad.bundle.js.map

This file was deleted.

2 changes: 1 addition & 1 deletion package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

5 changes: 3 additions & 2 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
"name": "@doist/reactist",
"description": "Open source React components by Doist",
"author": "Henning Muszynski <[email protected]> (http://doist.com)",
"version": "7.2.1",
"version": "9.0.0",
"license": "MIT",
"homepage": "https://github.com/Doist/reactist#readme",
"repository": "git+https://github.com/Doist/reactist.git",
Expand All @@ -21,7 +21,8 @@
"styles"
],
"engines": {
"node": ">=10"
"node": "^14.5.5",
"npm": "^6.14.11"
},
"scripts": {
"setup": "npm install && npm run validate",
Expand Down

0 comments on commit 3c008ba

Please sign in to comment.