Skip to content

Commit 52aeb52

Browse files
chore(devextreme): migrate compression-pipes.js (#33434)
Co-authored-by: Aliullov Vlad <vladislav.aliullov@devexpress.com> Co-authored-by: Aliullov Vlad <91639107+GoodDayForSurf@users.noreply.github.com>
1 parent 25cfbb7 commit 52aeb52

25 files changed

Lines changed: 911 additions & 545 deletions

nx.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,5 +45,6 @@
4545
"defaultBase": "master",
4646
"tui": {
4747
"enabled": false
48-
}
48+
},
49+
"analytics": false
4950
}

package.json

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,9 +29,9 @@
2929
"nx": {},
3030
"private": true,
3131
"devDependencies": {
32-
"@nx/devkit": "22.4.5",
33-
"@nx/jest": "22.4.5",
34-
"@nx/js": "22.4.5",
32+
"@nx/devkit": "22.7.0",
33+
"@nx/jest": "22.7.0",
34+
"@nx/js": "22.7.0",
3535
"@types/jest": "29.5.14",
3636
"@types/node": "20.12.8",
3737
"@types/shelljs": "0.8.15",
@@ -48,7 +48,7 @@
4848
"jest": "29.7.0",
4949
"jspdf-autotable": "3.8.4",
5050
"lint-staged": "14.0.1",
51-
"nx": "22.4.5",
51+
"nx": "22.7.0",
5252
"nx-cloud": "19.1.0",
5353
"postcss": "~8.5.10",
5454
"shelljs": "0.8.5",

packages/devextreme/build/gulp/aspnet.js

Lines changed: 0 additions & 18 deletions
This file was deleted.

packages/devextreme/build/gulp/compression-pipes.js

Lines changed: 0 additions & 82 deletions
This file was deleted.

packages/devextreme/build/gulp/js-bundles.js

Lines changed: 10 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -7,10 +7,10 @@ const named = require('vinyl-named');
77
const notify = require('gulp-notify');
88
const path = require('path');
99
const plumber = require('gulp-plumber');
10+
const shell = require('gulp-shell');
1011
const webpack = require('webpack');
1112
const webpackStream = require('webpack-stream');
1213

13-
const compressionPipes = require('./compression-pipes.js');
1414
const ctx = require('./context.js');
1515
const headerPipes = require('./header-pipes.js');
1616
const webpackConfig = require('../../webpack.config.js');
@@ -42,26 +42,11 @@ const getWebpackConfig = () => {
4242
return Object.assign(webpackConfig, { plugins });
4343
};
4444

45-
const bundleProdPipe = lazyPipe()
46-
.pipe(named)
47-
.pipe(() => webpackStream(getWebpackConfig(), webpack, muteWebPack))
48-
.pipe(headerPipes.useStrict)
49-
.pipe(headerPipes.bangLicense)
50-
.pipe(compressionPipes.minify);
51-
52-
const jsBundlesProd = (src, dist, bundles) => (() =>
53-
gulp.src(processBundles(bundles, src))
54-
.pipe(bundleProdPipe())
55-
.pipe(gulp.dest(dist))
56-
);
57-
58-
gulp.task('js-bundles-prod',
59-
jsBundlesProd(
60-
ctx.TRANSPILED_PROD_RENOVATION_PATH,
61-
ctx.RESULT_JS_PATH,
62-
BUNDLES,
63-
)
64-
);
45+
gulp.task('js-bundles-prod', shell.task(
46+
ctx.uglify
47+
? 'pnpm nx run devextreme:bundle:prod -c production'
48+
: 'pnpm nx run devextreme:bundle:prod'
49+
));
6550

6651
function prepareDebugMeta(watch) {
6752
const debugConfig = Object.assign({ watch }, getWebpackConfig());
@@ -94,16 +79,17 @@ function createDebugBundlesStream(watch, displayName) {
9479
.pipe(webpackStream(debugConfig, webpack, muteWebPack))
9580
.pipe(headerPipes.useStrict())
9681
.pipe(headerPipes.bangLicense())
97-
.pipe(gulpIf(!watch, compressionPipes.beautify()))
9882
.pipe(gulp.dest(destination));
9983

10084
task.displayName = `${displayName}-worker`;
10185

10286
return task;
10387
}
10488

105-
gulp.task('js-bundles-debug', gulp.series(
106-
createDebugBundlesStream(false, 'js-bundles-debug')
89+
gulp.task('js-bundles-debug', shell.task(
90+
ctx.uglify
91+
? 'pnpm nx run devextreme:bundle:debug -c production'
92+
: 'pnpm nx run devextreme:bundle:debug'
10793
));
10894

10995
gulp.task('js-bundles-watch', gulp.parallel(

packages/devextreme/build/gulp/npm.js

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,8 @@ const replace = require('gulp-replace');
1111
const lazyPipe = require('lazypipe');
1212
const gulpFilter = require('gulp-filter');
1313
const gulpRename = require('gulp-rename');
14+
const shell = require('gulp-shell');
1415

15-
const compressionPipes = require('./compression-pipes.js');
1616
const ctx = require('./context.js');
1717
const env = require('./env-variables.js');
1818
const dataUri = require('./gulp-data-uri').gulpPipe;
@@ -96,7 +96,6 @@ const sources = (src, dist, distGlob) => (() => merge(
9696
.src(src)
9797
.pipe(licenseValidator())
9898
.pipe(headerPipes.starLicense())
99-
.pipe(compressionPipes.beautify())
10099
.pipe(gulp.dest(dist)),
101100

102101
gulp
@@ -181,7 +180,12 @@ gulp.task('npm-sources', gulp.series(
181180
.src('../devextreme-dist/LICENSE.md')
182181
.pipe(gulp.dest(distPath)),
183182
),
184-
sources(srcGlobs, packagePath, distGlobs))
183+
sources(srcGlobs, packagePath, distGlobs),
184+
shell.task(
185+
ctx.uglify
186+
? 'pnpm nx run devextreme:compress:npm-sources -c production'
187+
: 'pnpm nx run devextreme:compress:npm-sources'
188+
))
185189
);
186190

187191
gulp.task('npm-dist', () => gulp

packages/devextreme/build/gulp/transpile.js

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,14 +5,16 @@ const gulp = require('gulp');
55
const notify = require('gulp-notify');
66
const path = require('path');
77
const plumber = require('gulp-plumber');
8+
const replace = require('gulp-replace');
89
const watch = require('gulp-watch');
910

10-
const removeDebug = require('./compression-pipes.js').removeDebug;
1111
const ctx = require('./context.js');
1212
const testsConfig = require('../../testing/tests.babelrc.json');
1313
const transpileConfig = require('./transpile-config');
1414
const createTsCompiler = require('./typescript/compiler');
1515

16+
const REMOVE_DEBUG_REGEXP = /\/{2,}\s{0,}#DEBUG[\s\S]*?\/{2,}\s{0,}#ENDDEBUG/g;
17+
1618
const src = [
1719
'js/**/*.*',
1820
'!js/**/*.d.ts',
@@ -48,7 +50,7 @@ const watchJsTask = () => {
4850
.pipe(babel(transpileConfig.cjs))
4951
.pipe(gulp.dest(ctx.TRANSPILED_PATH));
5052
watchTask
51-
.pipe(removeDebug())
53+
.pipe(replace(REMOVE_DEBUG_REGEXP, ''))
5254
.pipe(babel(transpileConfig.cjs))
5355
.pipe(gulp.dest(ctx.TRANSPILED_PROD_RENOVATION_PATH));
5456
return watchTask;
@@ -67,7 +69,7 @@ const watchTsTask = async() => {
6769
}))
6870
.pipe(babel(transpileConfig.tsCjs))
6971
.pipe(gulp.dest(ctx.TRANSPILED_PATH))
70-
.pipe(removeDebug())
72+
.pipe(replace(REMOVE_DEBUG_REGEXP, ''))
7173
.pipe(gulp.dest(ctx.TRANSPILED_PROD_RENOVATION_PATH));
7274
};
7375
watchTsTask.displayName = 'transpile TS watch';

packages/devextreme/build/gulp/ts.js

Lines changed: 22 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,10 @@ const footer = require('gulp-footer');
66
const concat = require('gulp-concat');
77
const path = require('path');
88
const replace = require('gulp-replace');
9+
const shell = require('gulp-shell');
910
const ts = require('gulp-typescript');
1011
const context = require('./context.js');
1112
const headerPipes = require('./header-pipes.js');
12-
const compressionPipes = require('./compression-pipes.js');
1313
const MODULES = require('./modules_metadata.json');
1414
const { packageDir } = require('./utils');
1515

@@ -105,26 +105,27 @@ gulp.task('ts-check-modules', function() {
105105
.pipe(compileTS());
106106
});
107107

108-
gulp.task('ts-copy-modules', function() {
109-
const BUNDLE_IMPORT = 'import DevExpress from \'../bundles/dx.all\';';
110-
111-
return gulp.src(src)
112-
/* legacy modules */
113-
.pipe(file('events/click.d.ts', BUNDLE_IMPORT))
114-
.pipe(file('events/contextmenu.d.ts', BUNDLE_IMPORT))
115-
.pipe(file('events/dblclick.d.ts', BUNDLE_IMPORT))
116-
.pipe(file('events/drag.d.ts', BUNDLE_IMPORT))
117-
.pipe(file('events/hold.d.ts', BUNDLE_IMPORT))
118-
.pipe(file('events/hover.d.ts', BUNDLE_IMPORT))
119-
.pipe(file('events/pointer.d.ts', BUNDLE_IMPORT))
120-
.pipe(file('events/swipe.d.ts', BUNDLE_IMPORT))
121-
.pipe(file('events/transform.d.ts', BUNDLE_IMPORT))
122-
.pipe(file('integration/jquery.d.ts', 'import \'jquery\';'))
123-
124-
.pipe(compressionPipes.removeDebug())
125-
.pipe(headerPipes.starLicense())
126-
.pipe(gulp.dest(packagePath));
127-
});
108+
gulp.task('ts-copy-modules', gulp.series(
109+
function tsCopyModulesCopy() {
110+
const BUNDLE_IMPORT = 'import DevExpress from \'../bundles/dx.all\';';
111+
112+
return gulp.src(src)
113+
/* legacy modules */
114+
.pipe(file('events/click.d.ts', BUNDLE_IMPORT))
115+
.pipe(file('events/contextmenu.d.ts', BUNDLE_IMPORT))
116+
.pipe(file('events/dblclick.d.ts', BUNDLE_IMPORT))
117+
.pipe(file('events/drag.d.ts', BUNDLE_IMPORT))
118+
.pipe(file('events/hold.d.ts', BUNDLE_IMPORT))
119+
.pipe(file('events/hover.d.ts', BUNDLE_IMPORT))
120+
.pipe(file('events/pointer.d.ts', BUNDLE_IMPORT))
121+
.pipe(file('events/swipe.d.ts', BUNDLE_IMPORT))
122+
.pipe(file('events/transform.d.ts', BUNDLE_IMPORT))
123+
.pipe(file('integration/jquery.d.ts', 'import \'jquery\';'))
124+
.pipe(headerPipes.starLicense())
125+
.pipe(gulp.dest(packagePath));
126+
},
127+
shell.task('pnpm nx run devextreme:compress:ts-modules')
128+
));
128129

129130
gulp.task('ts-sources', gulp.series('ts-copy-modules', 'ts-copy-bundle'));
130131

0 commit comments

Comments
 (0)