Skip to content
This repository has been archived by the owner on Dec 1, 2023. It is now read-only.

Commit

Permalink
Fix(hopp): use basename in stat cache (fix #51) (#56)
Browse files Browse the repository at this point in the history
* Fix(hopp): restrict use of : in task names

* Merge branch 'master' into fix_51
  • Loading branch information
karimsa authored Jul 6, 2017
1 parent 3f8c19e commit 2b19812
Show file tree
Hide file tree
Showing 22 changed files with 153 additions and 67 deletions.
2 changes: 1 addition & 1 deletion hopp.lock
Original file line number Diff line number Diff line change
@@ -1 +1 @@
{"v":"1.2.1","p":{"hopp-plugin-eslint":{}},"pl":[1499314442000,{"hopp-plugin-babel":["config","default"],"hopp-plugin-eslint":["config","format","failOnError","default"]}],"lp":{"hopp-plugin-babel":"/Users/karimsa/projects/hoppjs/hopp/packages/hopp-plugin-babel","hopp-plugin-eslint":"/Users/karimsa/projects/hoppjs/hopp/packages/hopp-plugin-eslint"},"_":[0,null],"sc":{"lint:lint:accord":{"./packages/hopp-plugin-accord/lib/index.js":1499314351000},"lint:lint:babel":{"./packages/hopp-plugin-babel/lib/index.js":1499314351000},"lint:lint:concat":{"./packages/hopp-plugin-concat/lib/index.js":1499314351000},"lint:lint:notify":{"./packages/hopp-plugin-notify/lib/index.js":1499314351000},"lint:lint:eslint":{"./packages/hopp-plugin-eslint/lib/index.js":1499314351000}}}
{"v":"1.2.1","p":{"hopp-plugin-eslint":{},"hopp-plugin-babel":{}},"pl":[1499316095000,{"hopp-plugin-babel":["config","default"],"hopp-plugin-eslint":["config","format","failOnError","default"]}],"lp":{"hopp-plugin-babel":"/Users/karimsa/projects/hoppjs/hopp/packages/hopp-plugin-babel","hopp-plugin-eslint":"/Users/karimsa/projects/hoppjs/hopp/packages/hopp-plugin-eslint"},"_":[0,null],"sc":{"lint-accord":{"./packages/hopp-plugin-accord/lib/index.js":1499316132000},"lint-babel":{"./packages/hopp-plugin-babel/lib/index.js":1499316132000},"lint-concat":{"./packages/hopp-plugin-concat/lib/index.js":1499316132000},"lint-notify":{"./packages/hopp-plugin-notify/lib/index.js":1499316132000},"lint-eslint":{"./packages/hopp-plugin-eslint/lib/index.js":1499316132000},"hopp-latest":{"./packages/hopp/src/cache.js":1499221452000,"./packages/hopp/src/compat/else.js":1499294007000,"./packages/hopp/src/fs/get-path.js":1499150171000,"./packages/hopp/src/fs/glob.js":1499314698000,"./packages/hopp/src/fs/index.js":1499221452000,"./packages/hopp/src/hopp.js":1499316095000,"./packages/hopp/src/hoppfile/find.js":1499221452000,"./packages/hopp/src/hoppfile/index.js":1499150171000,"./packages/hopp/src/hoppfile/load.js":1499221452000,"./packages/hopp/src/index.js":1499293987000,"./packages/hopp/src/streams/buffer.js":1499150171000,"./packages/hopp/src/streams/bundle.js":1499221452000,"./packages/hopp/src/streams/index.js":1499150171000,"./packages/hopp/src/streams/map.js":1499150171000,"./packages/hopp/src/streams/readstream.js":1499294007000,"./packages/hopp/src/tasks/goal.js":1499315614000,"./packages/hopp/src/tasks/loadPlugins.js":1499293987000,"./packages/hopp/src/tasks/mgr.js":1499316095000,"./packages/hopp/src/tasks/parallel.js":1499315659000,"./packages/hopp/src/tasks/steps.js":1499315684000,"./packages/hopp/src/tasks/tree.js":1499150171000,"./packages/hopp/src/tasks/watch.js":1499150171000,"./packages/hopp/src/utils/_.js":1499150171000,"./packages/hopp/src/utils/deep-equal.js":1499150171000,"./packages/hopp/src/utils/error.js":1499316095000,"./packages/hopp/src/utils/fn.js":1499221452000,"./packages/hopp/src/utils/index.js":1499316095000,"./packages/hopp/src/utils/log.js":1499221452000},"build-babel":{"./packages/hopp-plugin-babel/lib/index.js":1499314447000},"build-concat":{"./packages/hopp-plugin-concat/lib/index.js":1499314447000},"build-accord":{"./packages/hopp-plugin-accord/lib/index.js":1499314447000},"build-notify":{"./packages/hopp-plugin-notify/lib/index.js":1499314447000},"build-eslint":{"./packages/hopp-plugin-eslint/lib/index.js":1499316095000},"hopp-legacy":{"./packages/hopp/src/cache.js":1499221452000,"./packages/hopp/src/compat/else.js":1499294007000,"./packages/hopp/src/fs/get-path.js":1499150171000,"./packages/hopp/src/fs/glob.js":1499314698000,"./packages/hopp/src/fs/index.js":1499221452000,"./packages/hopp/src/hopp.js":1499316095000,"./packages/hopp/src/hoppfile/find.js":1499221452000,"./packages/hopp/src/hoppfile/index.js":1499150171000,"./packages/hopp/src/hoppfile/load.js":1499221452000,"./packages/hopp/src/index.js":1499293987000,"./packages/hopp/src/streams/buffer.js":1499150171000,"./packages/hopp/src/streams/bundle.js":1499221452000,"./packages/hopp/src/streams/index.js":1499150171000,"./packages/hopp/src/streams/map.js":1499150171000,"./packages/hopp/src/streams/readstream.js":1499294007000,"./packages/hopp/src/tasks/goal.js":1499315614000,"./packages/hopp/src/tasks/loadPlugins.js":1499293987000,"./packages/hopp/src/tasks/mgr.js":1499316095000,"./packages/hopp/src/tasks/parallel.js":1499315659000,"./packages/hopp/src/tasks/steps.js":1499315684000,"./packages/hopp/src/tasks/tree.js":1499150171000,"./packages/hopp/src/tasks/watch.js":1499150171000,"./packages/hopp/src/utils/_.js":1499150171000,"./packages/hopp/src/utils/deep-equal.js":1499150171000,"./packages/hopp/src/utils/error.js":1499316095000,"./packages/hopp/src/utils/fn.js":1499221452000,"./packages/hopp/src/utils/index.js":1499316095000,"./packages/hopp/src/utils/log.js":1499221452000}}}
18 changes: 9 additions & 9 deletions hoppfile.js
Original file line number Diff line number Diff line change
Expand Up @@ -29,14 +29,14 @@ const plugins = [
plugins.forEach(name => {
const shortName = name.substr(12)

exports['lint:' + shortName] =
exports['lint-' + shortName] =
hopp(`./packages/${name}/lib/**/**.js`)
.eslint({ fix: true })
.eslint.format()
.eslint.failOnError()
.dest()

exports['build:' + shortName] =
exports['build-' + shortName] =
hopp(`./packages/${name}/lib/**/**.js`)
.babel({
babelrc: false,
Expand All @@ -53,8 +53,8 @@ plugins.forEach(name => {

exports[shortName] =
hopp.steps([
'lint:' + shortName,
'build:' + shortName,
'lint-' + shortName,
'build-' + shortName,
])
})

Expand All @@ -76,12 +76,12 @@ function buildHopp(opts, dest) {
.dest('./packages/hopp/' + dest)
}

exports['hopp:latest'] =
exports['hopp-latest'] =
buildHopp({
plugins: ['transform-es2015-modules-commonjs']
}, 'dist')

exports['hopp:legacy'] =
exports['hopp-legacy'] =
buildHopp({
presets: [
['env', {
Expand All @@ -94,8 +94,8 @@ exports['hopp:legacy'] =

exports['hopp'] =
hopp.steps([
'hopp:latest',
'hopp:legacy'
'hopp-latest',
'hopp-legacy'
])

/**
Expand All @@ -106,7 +106,7 @@ exports.default = hopp.all(plugins.map(p => p.substr(12)).concat([ 'hopp' ]))
/**
* Global linting.
*/
exports.lint = hopp.all(plugins.map(name => 'lint:' + name.substr(12)))
exports.lint = hopp.all(plugins.map(name => 'lint-' + name.substr(12)))

/**
* Watch all.
Expand Down
63 changes: 33 additions & 30 deletions packages/hopp/dist-legacy/fs/glob.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion packages/hopp/dist-legacy/fs/glob.js.map

Large diffs are not rendered by default.

8 changes: 8 additions & 0 deletions packages/hopp/dist-legacy/tasks/goal.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion packages/hopp/dist-legacy/tasks/goal.js.map

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

8 changes: 8 additions & 0 deletions packages/hopp/dist-legacy/tasks/parallel.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit 2b19812

Please sign in to comment.