gulp-jspm icon indicating copy to clipboard operation
gulp-jspm copied to clipboard

Support jspm v0.17?

Open kscc25 opened this issue 9 years ago • 5 comments

jspm 0.17 beta is out for a while, I tried it today I got this:

Unhandled rejection TypeError: jspm.bundleSFX is now jspm.build
    at EventEmitter.API.bundleSFX (node_modules/jspm/api.js:116:25)
    at Promise.resolve.then.then.spread.then.results.temp_path (node_modules/gulp-jspm/index.js:42:68)

My gulp task configuration:

gulp.task('js', function () {
  gulp.src(['client/js/main.js'])
    .pipe(jspm({ selfExecutingBundle: true }))
    .pipe(gulp.dest('build/js/'));
});

kscc25 avatar Feb 12 '16 10:02 kscc25

I think the only change that is necessary would be to change this line. bundleSFX should simply be changed to build. I would also like to use this plugin with jspm 0.17 as it brings many improvements.

frederikschubert avatar Feb 14 '16 12:02 frederikschubert

up to do a PR? :)

brillout avatar Feb 14 '16 20:02 brillout

@brillout I think things are not simple like that, gulp-jspm is using its own jspm package to do the job but not the jspm package of the node application. This is not nice because we cannot support both JSPM 0.16 and 0.17 in the same time.

kscc25 avatar Feb 15 '16 02:02 kscc25

@heroandtn3 how about a beta version for gulp-jspm? beta gulp-jspm -> beta jspm, stable gulp-spm -> stable jspm

API changes that aren't backward compatible are a problem when using the jspm package of the application

brillout avatar May 02 '16 13:05 brillout

Sorry I have been traveling the last months and forgot to reply. The beta approach sounds good!

frederikschubert avatar May 02 '16 21:05 frederikschubert