Commit3e612ae

authored

Release: ensure builds have the proper version

- order hooks in execution order- update workflow actionsClosesgh-5519

1 parent1fa8df5 commit3e612aeCopy full SHA for 3e612ae

File tree

+33

-10

lines changed

+33

-10

lines changed

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -21,10 +21,10 @@ jobs:`
`21`	`21`	`NODE_VERSION:20.x`
`22`	`22`	`steps:`
`23`	`23`	`-name:Checkout`
`24`		`-uses:actions/checkout@a5ac7e51b41094c92402da3b24376905380afc29# v4.1.6`
	`24`	`+uses:actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332# v4.1.7`
`25`	`25`
`26`	`26`	`-name:Use Node.js ${{ env.NODE_VERSION }}`
`27`		`-uses:actions/setup-node@60edb5dd545a775178f52524783378180af0d1f8# v4.0.2`
	`27`	`+uses:actions/setup-node@1e60f620b9541d16bece96c5465dc8ee9832be0b# v4.0.3`
`28`	`28`	`with:`
`29`	`29`	`node-version:${{ env.NODE_VERSION }}`
`30`	`30`

Lines changed: 3 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -10,11 +10,11 @@ module.exports = {`
`10`	`10`	`preReleaseBase:1,`
`11`	`11`	`hooks:{`
`12`	`12`	`"before:init":"bash ./build/release/pre-release.sh",`
`13`		`-"before:git:release":"git add -f dist/ dist-module/ changelog.md",`
`14`	`13`	`"after:version:bump":`
`15`	`14`	`"sed -i 's/main\\/AUTHORS.txt/${version}\\/AUTHORS.txt/' package.json",`
`16`		`-"after:release":`
`17`		-`bash ./build/release/post-release.sh \${version}${blogURL}`
	`15`	`+"after:bump":"cross-env VERSION=${version} npm run build:all",`
	`16`	`+"before:git:release":"git add -f dist/ dist-module/ changelog.md",`
	`17`	+"after:release":`bash ./build/release/post-release.sh \${version}${blogURL}`
`18`	`18`	`},`
`19`	`19`	`git:{`
`20`	`20`	`changelog:"npm run release:changelog -- ${from} ${to}",`

Lines changed: 7 additions & 4 deletions

Original file line number	Diff line number	Diff line change
`@@ -24,13 +24,14 @@ const REGISTRY_URL = "https://registry.npmjs.org/jquery";`
`24`	`24`
`25`	`25`	`constrstable=/^(\d+\.\d+\.\d+)$/;`
`26`	`26`
`27`		`-exportasyncfunctionverifyRelease({ version}={}){`
	`27`	`+asyncfunctionverifyRelease({ version}={}){`
`28`	`28`	`if(!version){`
`29`	`29`	`version=process.env.VERSION\|\|(awaitgetLatestVersion());`
`30`	`30`	`}`
`31`		-console.log(`Checking jQuery${version}...`);
`32`	`31`	`constrelease=awaitbuildRelease({ version});`
`33`	`32`
	`33`	+console.log(`Verifying jQuery${version}...`);
	`34`	`+`
`34`	`35`	`letverified=true;`
`35`	`36`
`36`	`37`	`// Only check stable versions against the CDN`
`@@ -139,8 +140,8 @@ async function buildRelease( { version } ) {`
`139`	`140`	`.filter((dirent)=>dirent.isFile())`
`140`	`141`	`.map(async(dirent)=>({`
`141`	`142`	`name:dirent.name,`
`142`		`-path:path.basename(dirent.path),`
`143`		`-contents:awaitreadFile(path.join(dirent.path,dirent.name),"utf8")`
	`143`	`+path:path.basename(dirent.parentPath),`
	`144`	`+contents:awaitreadFile(path.join(dirent.parentPath,dirent.name),"utf8")`
`144`	`145`	`}))`
`145`	`146`	`);`
`146`	`147`
`@@ -196,3 +197,5 @@ async function sumTarball( filepath ) {`
`196`	`197`	`constunzipped=awaitgunzip(contents);`
`197`	`198`	`returnshasum(unzipped);`
`198`	`199`	`}`
	`200`	`+`
	`201`	`+verifyRelease();`

Lines changed: 19 additions & 0 deletions

Some generated files are not rendered by default. Learn more aboutcustomizing how changed files appear on GitHub.

Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -64,7 +64,7 @@`
`64`	`64`	`"release:changelog":"node build/release/changelog.js",`
`65`	`65`	`"release:clean":"rimraf tmp --glob changelog.{md,html} contributors.html",`
`66`	`66`	`"release:dist":"node build/release/dist.js",`
`67`		`-"release:verify":"node-e\"(async () => { const { verifyRelease } = await import('./build/release/verify.js'); verifyRelease() })()\"",`
	`67`	`+"release:verify":"node build/release/verify.js",`
`68`	`68`	`"start":"node -e\"(async () => { const { buildDefaultFiles } = await import('./build/tasks/build.js'); buildDefaultFiles({ watch: true }) })()\"",`
`69`	`69`	`"test:bundlers":"npm run pretest && npm run build:all && node test/bundler_smoke_tests/run-jsdom-tests.js",`
`70`	`70`	`"test:browser":"npm run pretest && npm run build:main && npm run test:unit -- -b chrome -b firefox -h",`
`@@ -119,6 +119,7 @@`
`119`	`119`	`"commitplease":"3.2.0",`
`120`	`120`	`"concurrently":"8.2.2",`
`121`	`121`	`"core-js-bundle":"3.37.1",`
	`122`	`+"cross-env":"7.0.3",`
`122`	`123`	`"diff":"5.2.0",`
`123`	`124`	`"eslint":"8.57.0",`
`124`	`125`	`"eslint-config-jquery":"3.0.2",`

Comments

(0)