Merge pull request #572 from smartprocure/feature/yarn-esbuild

ESM source, use yarn and esbuild

Author: stellarhoof

.babelrc

@@ -0,0 +1,29 @@
+{
+  "extends": [
+    "eslint:recommended",
+    "plugin:import/recommended",
+    "plugin:react/recommended"
+  ],
+  "ignorePatterns": ["dist", "node_modules", "!.storybook"],
+  "parserOptions": {
+    "ecmaVersion": "2022",
+    "sourceType": "module",
+    "ecmaFeatures": { "jsx": true }
+  },
+  "env": {
+    "browser": true,
+    "node": true,
+    "jest": true,
+    "es2022": true
+  },
+  "rules": {
+    "import/extensions": [2, { "js": "always" }],
+    "react/display-name": "off",
+    "react/prop-types": "off"
+  },
+  "settings": {
+    "react": {
+      "version": "detect"
+    }
+  }
+}

.eslintrc

@@ -1,36 +1,40 @@
-name: Pull Request
+name: Lint, and Danger
 
 on: pull_request
 
 jobs:
   build:
     runs-on: ubuntu-latest
     steps:
-    - uses: actions/checkout@v2
-      with:
-        token: '${{ secrets.PAT_DECRAPIFIER }}'
-        fetch-depth: 0
-    - name: Use Node.js 16.x
-      uses: actions/setup-node@v1
-      with:
-        node-version: 16.x
-    - name: Cache Node.js modules
-      uses: actions/cache@v2
-      with:
-        # npm cache files are stored in `~/.npm` on Linux/macOS
-        path: ~/.npm
-        key: ${{ runner.OS }}-node-${{ hashFiles('**/package-lock.json') }}
-        restore-keys: |
-          ${{ runner.OS }}-node-
-          ${{ runner.OS }}-
-    - run: npm run ci
-    - run: npm run build --if-present
-    - run: npm run test:ci || touch FAIL_JOB
-    - run: npm run lint:ci || touch FAIL_JOB
-    - name: pre-danger
-      run: git config user.email "decrapifier@govspend.com" && git config user.name "Decrapifier" && git checkout ${{github.head_ref}} && git config push.default upstream && git branch -u origin/${{github.head_ref}}
-    - run: npm run danger
-      env:
-        DANGER_GITHUB_API_TOKEN: '${{ secrets.PAT_DECRAPIFIER }}'
-    - name: check-if-lint-or-test-runs-failed
-      run: 'if test -f "FAIL_JOB"; then exit 1; fi'
+      - name: Checkout repo
+        uses: actions/checkout@v3
+        with:
+          token: ${{ secrets.PAT_DECRAPIFIER }}
+          fetch-depth: 0
+
+      - name: Setup node
+        uses: actions/setup-node@v3
+        with:
+          node-version: 16.x
+          cache: 'yarn'
+
+      - name: Install dependencies
+        run: yarn install --immutable
+
+      - name: Lint
+        run: yarn run lint:ci || touch FAIL_JOB
+
+      - name: Setup Decrapifier user
+        run: |
+          git config user.email "decrapifier@govspend.com"
+          git config user.name "Decrapifier"
+          git config push.default upstream
+          git checkout --track origin/${{github.head_ref}}
+
+      - name: Run danger
+        run: yarn run danger ci || touch FAIL_JOB
+        env:
+          DANGER_GITHUB_API_TOKEN: ${{ secrets.PAT_DECRAPIFIER }}
+
+      - name: check-if-lint-or-runs-failed
+        run: 'if test -f "FAIL_JOB"; then exit 1; fi'

.eslintrc.json

@@ -1,32 +1,39 @@
-name: Push to master
+name: Release
 
 on:
   push:
     branches:
       - master
+
 jobs:
   build:
     runs-on: ubuntu-latest
     steps:
-    - uses: actions/checkout@v2
-    - name: Use Node.js 16.x
-      uses: actions/setup-node@v1
-      with:
-        node-version: 16.x
-    - run: npm run ci
-    - run: npm run build --if-present
-    - run: npm test
-    - name: Publish to npm
-      run: |
-        echo "//registry.npmjs.org/:_authToken=$NPM_TOKEN" >> ~/.npmrc
-        npm publish
-      env:
-        NPM_TOKEN: '${{ secrets.NPM_TOKEN }}'
-    - name: Update package to grey-vest
-      run: npm run update-pkg
-    - name: Publish to npm
-      run: |
-        echo "//registry.npmjs.org/:_authToken=$NPM_TOKEN" >> ~/.npmrc
-        npm publish
-      env:
-        NPM_TOKEN: '${{ secrets.NPM_TOKEN }}'
+      - name: Checkout repo
+        uses: actions/checkout@v3
+
+      - name: Setup node
+        uses: actions/setup-node@v3
+        with:
+          node-version: 16.x
+          cache: 'yarn'
+
+      - name: Install dependencies
+        run: yarn install --immutable
+
+      - name: Publish to npm
+        run: |
+          echo -e "npmRegistryServer: \"https://registry.npmjs.org/\"\nnpmAuthToken: \"$NPM_TOKEN\"" >> ./.yarnrc.yml
+          yarn npm publish --access public
+        env:
+          NPM_TOKEN: '${{ secrets.NPM_TOKEN }}'
+
+      - name: Update package to grey-vest
+        run: yarn run update-pkg
+
+      - name: Publish to npm
+        run: |
+          echo -e "npmRegistryServer: \"https://registry.npmjs.org/\"\nnpmAuthToken: \"$NPM_TOKEN\"" >> ./.yarnrc.yml
+          yarn npm publish --access public
+        env:
+          NPM_TOKEN: '${{ secrets.NPM_TOKEN }}'

.github/workflows/pull-request.yaml

@@ -57,16 +57,20 @@ typings/
 # dotenv environment variables file
 .env
 
-# jest cpverage
-coverage/
-jest_0/
-
 # ignore distribution folder
-dist/
+dist/*
 
 # ignore storybook related folders
 storybook-static/
 
 test-results.json
-
-.idea/
+lint-results.json
+
+# yarn stuff
+# see https://yarnpkg.com/getting-started/qa#which-files-should-be-gitignored
+.yarn/*
+!.yarn/patches
+!.yarn/plugins
+!.yarn/releases
+!.yarn/sdks
+!.yarn/versions

.github/workflows/push-master.yaml

@@ -1,2 +1,2 @@
-import '@storybook/addon-actions/register'
-import '@storybook/addon-docs/register'
+import '@storybook/addon-actions/register.js'
+import '@storybook/addon-docs/register.js'

.gitignore

@@ -0,0 +1,2 @@
+nodeLinker: node-modules
+yarnPath: .yarn/releases/yarn-3.3.1.cjs

.npmignore

@@ -1,3 +1,8 @@
+### 2.54.9
+* Use yarn
+* Use eslint to build source
+* Convert package to ESM
+
 ### 2.54.8
 * FilterList - Add `resultsPath` similar to how it is used in TagsQuerySearchBar to indicate if we need to trigger tree update
 

.storybook/addons.js

@@ -0,0 +1,45 @@
+let { danger, markdown, fail, message, warn } = require('danger')
+
+let fs = require('fs')
+let duti = require('duti')
+
+let readJson = path => {
+  try {
+    return JSON.parse(fs.readFileSync(path, { encoding: 'utf8' }))
+  } catch (e) {
+    //
+  }
+}
+
+let args = {
+  danger,
+  fail,
+  message,
+  warn,
+  markdown,
+  lintResults: readJson('./lint-results.json'),
+  config: {
+    prNetChangeThreshold: 500,
+    personalityNetChangeThreshold: 500,
+    recommendedPrReviewers: 1,
+    rootFolder: 'src',
+  },
+}
+
+// Danger can do a run on a local repo via `danger local`, in which case
+// `danger.github` will not be defined:
+// - `danger local -h`
+// - https://danger.systems/js/tutorials/fast-feedback.html
+if (danger.github) {
+  duti.prAssignee(args)
+  duti.netNegativePR(args)
+  duti.bigPr(args)
+  duti.noPrDescription(args)
+  duti.requestedReviewers(args)
+  duti.emptyChangelog(args)
+  duti.versionBump(args)
+  duti.autoFix(args)
+}
+
+duti.hasLintWarnings(args)
+duti.hasLintErrors(args)