-
Notifications
You must be signed in to change notification settings - Fork 511
Fix Package
task
#3439
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Fix Package
task
#3439
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
e18f5ee
to
656fb35
Compare
656fb35
to
b457f4e
Compare
@rjmholt Running a "release" to see if everything still works. |
andyleejordan
commented
Jun 25, 2021
andyleejordan
commented
Jun 25, 2021
andyleejordan
commented
Jun 25, 2021
Oof, ok so this doesn't quite work. It's a chicken-egg problem. We need |
b457f4e
to
4bcbdb2
Compare
andyleejordan
commented
Jun 25, 2021
ecfca5f
to
d58815d
Compare
This now leaves the VSIX named as `vsce` creates it: `powershell-YYYY.MM.X.vsix`, instead of renaming it `PowerShell-Insiders.vsix`. I can't think of anything this would break, since the upload is not actually done with the deleted `Upload` task but with an ADO pipeline task.
This fixes the path to the script which is now part of the `PowerShellEditorServices` repository. This was necessary in order to sign it, as it must be signed _before_ the VSIX is created. Moving it to the other repo was the best way to solve the "chicken-egg" problem that presented.
d58815d
to
00da48b
Compare
@andys-mac-mini ~/src/vscode-powershell andschwa/package-fix ≡
> . '/Users/andschwa/.vscode/extensions/ms-vscode.powershell-2021.6.2/modules/PowerShellEditorServices/InvokePesterStub.ps1' -ScriptPath '/Users/andschwa/src/PowerShellEditorServices/test/PowerShellEditorServices.Test.Shared/Symbols/PesterFile.tests.ps1' -All -Output 'FromPreference'
Importing Pester module...
WARNING: Failed to import Pester. You must install Pester module to run or debug Pester tests.
WARNING: You can install Pester by executing: Install-Module Pester -Scope CurrentUser -Force
PS> Install-Module Pester -Scope CurrentUser -Force
@andys-mac-mini ...itorServices.Test.Shared/Symbols andschwa/import-pester-stub ≡ > . '/Users/andschwa/.vscode/extensions/ms-vscode.powershell-2021.6.2/modules/PowerShellEditorServices/InvokePesterStub.ps1' -ScriptPath '/Users/andschwa/src/PowerShellEditorServices/test/PowerShellEditorServices.Test.Shared/Symbols/PesterFile.tests.ps1' -All -Output 'FromPreference'
Importing Pester module...
Starting discovery in 1 files.
Discovery finished in 188ms.
Running tests.
[+] /Users/andschwa/src/PowerShellEditorServices/test/PowerShellEditorServices.Test.Shared/Symbols/PesterFile.tests.ps1 507ms (33ms|317ms)
Tests completed in 528ms
Tests Passed: 3, Failed: 0, Skipped: 0 NotRun: 0
PS> |
rjmholt
approved these changes
Jun 28, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This now leaves the VSIX named as
vsce
creates it:powershell-YYYY.MM.X.vsix
, instead of renaming itPowerShell-Insiders.vsix
.I can't think of anything this would break, since the upload is not actually
done with the deleted
Upload
task but with an ADO pipeline task.This also fixes the path to
InvokePesterStub.ps1
which is now part of thePowerShellEditorServices repository. This was necessary in order to sign it, as
it must be signed before the VSIX is created. Moving it to the other repo was
the best way to solve the "chicken-egg" problem that presented.
Resolves #3293 and combined with PowerShell/PowerShellEditorServices#1513, resolves #2536.