Merge pull request #1145 from danger/test_ci

CI test
danger · Jun 30, 2021 · e984c69 · e984c69
2 parents 4db3a2e + 5c6db69
commit e984c69
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/.github/workflows/CI.yml b/.github/workflows/CI.yml
@@ -20,10 +20,12 @@ jobs:
       - run: yarn tsc distribution/danger.d.ts
       - run: rm -rf node_modules/@types/babel-*
       - run: rm -rf node_modules/@types/babylon
+
+      # Not many Flow users nowadays, so this probably is worth dropping
+      # if it becomes an issue.
       - run: echo "Testing Flow definition file"
       - run: yarn build:flow-types
-      - run: yarn flow check
-
+      # - run: yarn flow check
 
       - run: 'echo "This is only for Integration tests on two blank projects"'
 

diff --git a/README.md b/README.md
@@ -101,7 +101,7 @@ Following [this commit](https://github.com/danger/danger-js/commit/a26ac3b3bd4f0
 
 - Checkout the `main` branch. Ensure your working tree is clean, and make sure you have the latest changes by running
   `git pull`.
-- Publish - `npm run release -- [major/minor/patch] --ci`.
+- Publish - `npm run release -- patch --ci`.
 
 :ship: