Skip to content

Commit

Permalink
Remove kcheck (babel#173)
Browse files Browse the repository at this point in the history
  • Loading branch information
danez authored and Kristof degrave committed Oct 27, 2016
1 parent 3848beb commit f4ddf16
Show file tree
Hide file tree
Showing 7 changed files with 50 additions and 23 deletions.
5 changes: 0 additions & 5 deletions .eslintignore

This file was deleted.

6 changes: 6 additions & 0 deletions .eslintrc
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
{
"extends": "babel",
"env": {
"node": true
}
}
12 changes: 12 additions & 0 deletions .flowconfig
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
[ignore]
.*/lib/.*
.*/test/.*

[include]

[libs]

[options]
strip_root=true
suppress_comment= \\(.\\|\n\\)*\\$FlowFixMe
suppress_comment= \\(.\\|\n\\)*\\$FlowIssue
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -3,3 +3,4 @@ build
coverage
lib
node_modules
yarn.lock
33 changes: 20 additions & 13 deletions .travis.yml
Original file line number Diff line number Diff line change
@@ -1,24 +1,31 @@
sudo: false
language: node_js
before_install:
- |
[ "$TRAVIS_COMMIT_RANGE" = "" ] || git diff --name-only $TRAVIS_COMMIT_RANGE | grep -qvE '(\.md$)|(^(doc))/' || {
echo "Only docs were updated, stopping build process."
exit
}
node_js:
- "6"
- "5"
- "4"
- "0.10"
- "0.12"
- "4"
- "5"
- "6"

before_script: make bootstrap-babel
before_script: 'if [ -n "${BABEL-}" ]; then make bootstrap-babel ; fi'

script:
- npm run test_cov
- make test-babel
- 'if [ -n "${LINT-}" ]; then npm run lint ; fi'
- 'if [ -n "${FLOW-}" ]; then npm run flow ; fi'
- 'if [ -n "${BABEL-}" ]; then make test-babel ; fi'
- 'if [ -z "${LINT-}" ] && [ -z "${FLOW-}" ] && [ -z "${BABEL-}" ]; then npm run test-ci ; fi'

matrix:
fast_finish: true
include:
- node_js: "node"
env: LINT=true
- node_js: "node"
env: FLOW=true
- node_js: "node"
env: BABEL=true

after_success: npm run coverage
after_success: 'if [ -z "${LINT-}" ] && [ -z "${FLOW-}" ] && [ -z "${BABEL-}" ]; then npm run coverage ; fi'

notifications:
slack: babeljs:5Wy4QX13KVkGy9CnU0rmvgeK
14 changes: 10 additions & 4 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,14 +14,19 @@
"devDependencies": {
"ava": "^0.16.0",
"babel-cli": "^6.14.0",
"babel-eslint": "^7.0.0",
"babel-helper-fixtures": "^6.9.0",
"babel-plugin-istanbul": "^2.0.1",
"babel-plugin-transform-flow-strip-types": "^6.14.0",
"babel-preset-es2015": "^6.14.0",
"babel-preset-stage-0": "^6.5.0",
"codecov": "^1.0.1",
"cross-env": "^2.0.1",
"kcheck": "^2.0.3",
"eslint": "^3.7.1",
"eslint-config-babel": "^2.0.1",
"eslint-plugin-babel": "^3.3.0",
"eslint-plugin-flowtype": "^2.20.0",
"flow-bin": "^0.33.0",
"lodash": "^4.15.0",
"nyc": "^8.1.0",
"unicode-9.0.0": "~0.7.0"
Expand All @@ -32,12 +37,13 @@
"scripts": {
"build": "babel src --out-dir lib",
"coverage": "nyc report --reporter=json && codecov -f coverage/coverage-final.json",
"lint": "kcheck",
"lint": "eslint src bin",
"flow": "flow",
"prepublish": "cross-env BABEL_ENV=production npm run build",
"preversion": "npm run test && npm run changelog",
"test": "npm run build && npm run lint && npm run test-only",
"test": "npm run lint && npm run flow && npm run build && npm run test-only",
"test-only": "ava test",
"test_cov": "cross-env BABEL_ENV=test npm run build && npm run lint && nyc ava test",
"test-ci": "cross-env BABEL_ENV=test npm run build && nyc npm run test-only",
"watch": "babel src --out-dir lib --watch",
"changelog": "git log `git describe --tags --abbrev=0`..HEAD --pretty=format:' * %s (%an)' | grep -v 'Merge pull request'"
},
Expand Down
2 changes: 1 addition & 1 deletion src/parser/statement.js
Original file line number Diff line number Diff line change
Expand Up @@ -419,7 +419,7 @@ pp.parseEmptyStatement = function (node) {
};

pp.parseLabeledStatement = function (node, maybeName, expr) {
for (let label of (this.state.labels: Array<Object>)){
for (let label of (this.state.labels: Array<Object>)) {
if (label.name === maybeName) {
this.raise(expr.start, `Label '${maybeName}' is already declared`);
}
Expand Down

0 comments on commit f4ddf16

Please sign in to comment.