javascript/ionic-framework
1
0
Fork 0
mirror of https://github.com/ionic-team/ionic-framework.git synced 2025-08-20 12:29:55 +08:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #6155 from driftyco/add-lint-karma-to-release

Browse Source 
chore(build): add tslint and unit tests to prerelease gulp task
This commit is contained in:
Adam Bradley
2016-04-13 20:03:34 -05:00
parent 0bdeab62e2 ade93371a4
commit 4ba9eb0807
3 changed files with 28 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

36
gulpfile.js
View File

@ -689,9 +689,16 @@ function buildDemoSass(isProductionMode) {
require('./scripts/snapshot/snapshot.task')(gulp, argv, buildConfig);
// requires bundle.system to be run once
gulp.task('karma', ['tests'], function() {
gulp.task('karma', ['tests'], function(done) {
var karma = require('karma').server;
return karma.start({ configFile: __dirname + '/scripts/karma/karma.conf.js' })
karma.start({
configFile: __dirname + '/scripts/karma/karma.conf.js'
}, function(result) {
if (result > 0) {
return done(new Error('Karma exited with an error'));
}
done();
});
});
gulp.task('karma-watch', ['watch.tests', 'bundle.system'], function() {
@ -710,8 +717,15 @@ gulp.task('karma-watch', ['watch.tests', 'bundle.system'], function() {
* some prerelease magic (see 'prepare') and copies npm package and tooling
* files to dist.
*/
gulp.task('prerelease', ['prepare', 'build.release'], function(done){
runSequence('package', done);
gulp.task('prerelease', function(done){
runSequence(
'tslint',
'prepare',
'build.release',
'karma',
'package',
done
);
});
/**
@ -931,11 +945,11 @@ gulp.task('tooling', function(){
/**
* TS LINT
*/
gulp.task("tslint", function() {
var tslint = require("gulp-tslint");
gulp.src([
'ionic/**/*.ts',
'!ionic/**/test/**/*',
]).pipe(tslint())
.pipe(tslint.report('verbose'));
gulp.task('tslint', function(done) {
var tslint = require('gulp-tslint');
return gulp.src([
'ionic/**/*.ts',
'!ionic/**/test/**/*',
]).pipe(tslint())
.pipe(tslint.report('verbose'));
});

2
ionic/components/img/img.ts
View File

@ -84,7 +84,7 @@ export class Img {
}
private _loaded(isLoaded: boolean) {
this._elementRef.nativeElement.classList[isLoaded ? 'add': 'remove']('img-loaded');
this._elementRef.nativeElement.classList[isLoaded ? 'add' : 'remove']('img-loaded');
}
enable(shouldEnable: boolean) {

4
ionic/platform/platform.ts
View File

@ -190,7 +190,7 @@ export class Platform {
triggerReady() {
this._zone.run(() => {
this._readyResolve();
})
});
}
/**
@ -741,5 +741,5 @@ export interface PlatformVersion {
str?: string;
num?: number;
major?: number;
minor?: number
minor?: number;
}