Skip to content

Commit 8d21dce

Browse files
authored
Merge branch 'jquery:main' into master
2 parents b58cd30 + 349ee77 commit 8d21dce

File tree

4 files changed

+65
-15
lines changed

4 files changed

+65
-15
lines changed

.npmignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
demos
2+
/dist/cdn
23
tests
34
.editorconfig
45
.eslintrc.json

AUTHORS.txt

Lines changed: 32 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -331,3 +331,35 @@ Peter Dave Hello <hsu@peterdavehello.org>
331331
Johannes Schäfer <johnschaefer@gmx.de>
332332
Ville Skyttä <ville.skytta@iki.fi>
333333
Ryan Oriecuia <ryan.oriecuia@visioncritical.com>
334+
Sergei Ratnikov <sergeir82@gmail.com>
335+
milk54 <milk851@gmail.com>
336+
Evelyn Masso <evoutofambit@gmail.com>
337+
Robin <mail@robin-fowler.com>
338+
Simon Asika <asika32764@gmail.com>
339+
Kevin Cupp <kevin.cupp@gmail.com>
340+
Jeremy Mickelson <Jeremy.Mickelson@gmail.com>
341+
Kyle Rosenberg <kyle.rosenberg@gmail.com>
342+
Petri Partio <petri.partio@gmail.com>
343+
pallxk <github@pallxk.com>
344+
Luke Brookhart <luke@onjax.com>
345+
claudi <hirt-claudia@gmx.de>
346+
Eirik Sletteberg <eiriksletteberg@gmail.com>
347+
Albert Johansson <albert@intervaro.se>
348+
A. Wells <borgboyone@users.noreply.github.com>
349+
Robert Brignull <robertbrignull@gmail.com>
350+
Horus68 <pauloizidoro@gmail.com>
351+
Maksymenkov Eugene <foatei@gmail.com>
352+
OskarNS <soerensen.oskar@gmail.com>
353+
Gez Quinn <holla@gezquinn.design>
354+
jigar gala <jigar.gala140291@gmail.com>
355+
Florian Wegscheider <flo.wegscheider@gmail.com>
356+
Fatér Zsolt <fater.zsolt@gmail.com>
357+
Szabolcs Szabolcsi-Toth <nec@shell8.net>
358+
Jérémy Munsch <github@jeremydev.ovh>
359+
Hrvoje Novosel <hrvoje.novosel@gmail.com>
360+
Paul Capron <PaulCapron@users.noreply.github.com>
361+
Micah Miller <mikhey@runbox.com>
362+
sakshi87 <53863764+sakshi87@users.noreply.github.com>
363+
Mikolaj Wolicki <wolicki.mikolaj@gmail.com>
364+
Patrick McKay <patrick.mckay@vumc.org>
365+
c-lambert <58025159+c-lambert@users.noreply.github.com>

build/release-test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ script( Release );
3737

3838
// Ignores actual version installed, should be good enough for a test
3939
if ( shell.exec( "npm ls --depth 0 | grep download.jqueryui.com" ).code === 1 ) {
40-
shell.exec( "npm install " + script.dependencies.join( " " ) );
40+
shell.exec( "npm install --no-save " + script.dependencies.join( " " ) );
4141
}
4242

4343
// If AUTHORS.txt is outdated, this will update it

build/release.js

Lines changed: 31 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -59,8 +59,21 @@ function addManifest( packager ) {
5959
function buildCDNPackage( callback ) {
6060
console.log( "Building CDN package" );
6161
var JqueryUi = require( "download.jqueryui.com/lib/jquery-ui" );
62-
var Package = require( "download.jqueryui.com/lib/package-1-12-themes" );
62+
var PackageWithoutThemes = require( "download.jqueryui.com/lib/package-1-13" );
63+
var PackageOfThemes = require( "download.jqueryui.com/lib/package-1-13-themes" );
6364
var Packager = require( "node-packager" );
65+
66+
// PackageOfThemes doesn't contain JS files, PackageWithoutThemes doesn't contain themes;
67+
// we need both.
68+
function Package() {
69+
70+
// PackageOfThemes invokes PackageWithoutThemes's constructor in its own so we don't
71+
// need to do it by ourselves; we just need to handle prototypes that way.
72+
PackageOfThemes.apply( this, arguments );
73+
}
74+
75+
Object.assign( Package.prototype, PackageWithoutThemes.prototype, PackageOfThemes.prototype );
76+
6477
var jqueryUi = new JqueryUi( path.resolve( "." ) );
6578
var target = fs.createWriteStream( "../" + jqueryUi.pkg.name + "-" + jqueryUi.pkg.version +
6679
"-cdn.zip" );
@@ -71,18 +84,22 @@ function buildCDNPackage( callback ) {
7184
jqueryUi: jqueryUi,
7285
themeVars: null
7386
} );
74-
packager.ready.then( function() {
75-
removeExternals( packager );
76-
addManifest( packager );
77-
packager.toZip( target, {
78-
basedir: ""
79-
}, function( error ) {
80-
if ( error ) {
81-
Release.abort( "Failed to zip CDN package", error );
82-
}
83-
callback();
87+
packager.ready
88+
.then( function() {
89+
removeExternals( packager );
90+
addManifest( packager );
91+
packager.toZip( target, {
92+
basedir: ""
93+
}, function( error ) {
94+
if ( error ) {
95+
Release.abort( "Failed to zip the CDN package", error );
96+
}
97+
callback();
98+
} );
99+
} )
100+
.catch( function( error ) {
101+
Release.abort( "Failed to create the CDN package", error );
84102
} );
85-
} );
86103
}
87104

88105
Release.define( {
@@ -123,7 +140,7 @@ Release.define( {
123140
};
124141

125142
module.exports.dependencies = [
126-
"download.jqueryui.com@2.1.2",
143+
"download.jqueryui.com@2.2.1",
127144
"node-packager@0.0.6",
128-
"shelljs@0.2.6"
145+
"shelljs@0.8.4"
129146
];

0 commit comments

Comments
 (0)