Merge branch 'remove-danger' into 'develop'
Remove Danger CI See merge request soapbox-pub/soapbox-fe!1543
This commit is contained in:
commit
c61e7ac888
4 changed files with 11 additions and 625 deletions
|
@ -22,13 +22,6 @@ deps:
|
||||||
changes:
|
changes:
|
||||||
- yarn.lock
|
- yarn.lock
|
||||||
|
|
||||||
danger:
|
|
||||||
stage: test
|
|
||||||
script: npx danger ci
|
|
||||||
cache: []
|
|
||||||
only:
|
|
||||||
- merge_requests
|
|
||||||
|
|
||||||
lint-js:
|
lint-js:
|
||||||
stage: test
|
stage: test
|
||||||
script: yarn lint:js
|
script: yarn lint:js
|
||||||
|
|
|
@ -1,13 +0,0 @@
|
||||||
import { danger, warn, message } from 'danger';
|
|
||||||
|
|
||||||
const docs = danger.git.fileMatch('docs/**/*.md');
|
|
||||||
const app = danger.git.fileMatch('app/**/*.(js|ts|tsx)');
|
|
||||||
const tests = danger.git.fileMatch('*/__tests__/*');
|
|
||||||
|
|
||||||
if (docs.edited) {
|
|
||||||
message('Thanks - We :heart: our [documentarians](http://www.writethedocs.org/)!');
|
|
||||||
}
|
|
||||||
|
|
||||||
if (app.modified && !tests.modified) {
|
|
||||||
warn('You have app changes without tests.');
|
|
||||||
}
|
|
|
@ -209,7 +209,6 @@
|
||||||
"babel-eslint": "^10.1.0",
|
"babel-eslint": "^10.1.0",
|
||||||
"babel-jest": "^28.1.0",
|
"babel-jest": "^28.1.0",
|
||||||
"cross-env": "^7.0.3",
|
"cross-env": "^7.0.3",
|
||||||
"danger": "^11.0.7",
|
|
||||||
"eslint": "^7.0.0",
|
"eslint": "^7.0.0",
|
||||||
"eslint-plugin-import": "^2.25.4",
|
"eslint-plugin-import": "^2.25.4",
|
||||||
"eslint-plugin-jsdoc": "^39.2.9",
|
"eslint-plugin-jsdoc": "^39.2.9",
|
||||||
|
|
Loading…
Reference in a new issue