Skip to content

Commit

Permalink
Ensure signing/packaging works as expected (#4088)
Browse files Browse the repository at this point in the history
* test out signing js and packaging it without messing up sign

* yarn compile-production

* temp image override

* confirmed, this fixes signing
  • Loading branch information
gcampbell-msft authored Sep 20, 2024
1 parent 8d35cf5 commit 3305ff2
Show file tree
Hide file tree
Showing 4 changed files with 1 addition and 8 deletions.
3 changes: 0 additions & 3 deletions jobs/release/prerelease.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,9 +29,6 @@ extends:
templateContext:
type: releaseJob
isProduction: true
sdl:
codeSignValidation:
additionalTargetsGlobPattern: -|**\*.js
inputs:
- input: pipelineArtifact
pipeline: BuildPrereleasePipeline
Expand Down
3 changes: 0 additions & 3 deletions jobs/release/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -61,9 +61,6 @@ extends:
templateContext:
type: releaseJob
isProduction: true
sdl:
codeSignValidation:
additionalTargetsGlobPattern: -|**\*.js
inputs:
- input: pipelineArtifact
pipeline: BuildReleasePipeline
Expand Down
2 changes: 1 addition & 1 deletion jobs/shared/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ steps:
displayName: Build files
inputs:
script: |
npm run vscode:prepublish
yarn compile-production
- task: MSBuild@1
displayName: Sign files
inputs:
Expand Down
1 change: 0 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -3677,7 +3677,6 @@
"@test": "out/test/"
},
"scripts": {
"vscode:prepublish": "yarn run compile-production",
"compile": "yarn install && webpack --mode development",
"compile-watch": "yarn install && webpack --mode development --watch --progress",
"compile-production": "yarn install && yarn run translations-generate && webpack --env BUILD_VSCODE_NLS=true --mode production",
Expand Down

0 comments on commit 3305ff2

Please sign in to comment.