fix: Better workflow

Author: tannerlinsley

.github/workflows/publish.yml

@@ -1,4 +1,5 @@
-name: react-charts publish
+name: react-charts tests
+
 on:
   push:
     branches:
@@ -8,38 +9,35 @@ on:
 
 jobs:
   test:
-    name: 'node ${{ matrix.node }} ${{ matrix.os }} '
-    runs-on: '${{ matrix.os }}'
+    name: 'Node ${{ matrix.node }}'
+    runs-on: ubuntu-latest
     strategy:
       matrix:
-        os: [ubuntu-latest]
-        node: [12, 14, 16]
+        node: [10, 12, 14, 16]
     steps:
-      - uses: actions/setup-node@v1
-        with:
-          node-version: ${{ matrix.node }}
       - uses: actions/checkout@v2
+      - uses: actions/setup-node@v2
         with:
-          fetch-depth: 1
-      - run: npm i -g yarn
-      - run: yarn --frozen-lockfile
+          node-version: ${{ matrix.node }}
+      - name: Install dependencies
+        uses: bahmutov/npm-install@v1
       - run: yarn test:ci
 
   publish-module:
     name: 'Publish Module to NPM'
-    # needs: test
-    if: github.ref == 'refs/heads/main' || github.ref == 'refs/heads/beta' #publish only when merged in master, not on PR
+    needs: test
+    # publish only when merged in master on original repo, not on PR
+    if: github.repository == 'tannerlinsley/react-charts' && (github.ref == 'refs/heads/main' || github.ref == 'refs/heads/beta')
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
+      - uses: actions/setup-node@v2
         with:
-          fetch-depth: 1
-      - uses: actions/setup-node@v1
-        with:
-          node-version: 12
+          node-version: 14
           registry-url: https://registry.npmjs.org/
-      - run: npm i -g yarn
-      - run: yarn --frozen-lockfile
+      - name: Install dependencies
+        uses: bahmutov/npm-install@v1
+      - run: yarn build
       - run: npx semantic-release@17
         env:
           NODE_AUTH_TOKEN: ${{secrets.npm_token}}

.gitignore

@@ -1,27 +1,35 @@
-.cache
-.DS_Store
+
+# See https://help.github.com/ignore-files/ for more about ignoring files.
+
+# dependencies
+node_modules
+
+# builds
+types
+build
+dist
+lib
+es
+artifacts
+.rpt2_cache
+coverage
+*.tgz
+
+# misc
 .DS_Store
 .env
-.env.development.local
 .env.local
-.env.production.local
+.env.development.local
 .env.test.local
-.history
+.env.production.local
 .next
-.rpt2_cache
-*.log
-artifacts
-build
-d3/index.js
-dist
-dist
-node_modules
-node_modules
+
 npm-debug.log*
-size-plugin.json
-stats.html
 yarn-debug.log*
 yarn-error.log*
-es
-lib
-types
+.history
+size-plugin.json
+stats-hydration.json
+stats-react.json
+stats.html
+.vscode/settings.json

package.json

@@ -1,7 +1,7 @@
 {
   "name": "react-charts",
   "version": "1.0.0-semantic-release",
-  "description": "Hooks for managing, caching and syncing asynchronous and remote data in React",
+  "description": "Simple, immersive & interactive charts for React. Flexible, declarative, and highly configurable.",
   "author": "tannerlinsley",
   "license": "MIT",
   "repository": "tannerlinsley/react-charts",