-
Notifications
You must be signed in to change notification settings - Fork 35
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
Builtins v1.64.2 #106
Builtins v1.64.2 #106
Conversation
…versions Like so: "preview": false Signed-off-by: Marc Dumais <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have not tested it yet,
The code looks good, Just a cosmetic update on a reference,
See inline comment.
src/create-extension-pack.js
Outdated
@@ -51,7 +51,7 @@ const repository = 'https://github.com/eclipse-theia/vscode-builtin-extensions'; | |||
* fetched by vscode at build time. | |||
* https://github.com/microsoft/vscode/blob/1.57.0/product.json#L34-L126 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The reference needs to be updated to 1.64.2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done. I replace the URL with a URI that points to the version of product.json
that's checked-out locally.
5f0c0f0
to
fccfdda
Compare
Signed-off-by: Marc Dumais <[email protected]> ran yarn upgrade, to update the repo's dependencies Signed-off-by: Marc Dumais <[email protected]>
fccfdda
to
cbe803e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks good to me as well!
Thanks Marc !!
Proceeding with release! |
In preparation of publishing the vscode builtin extensions for 1.64.2. Also include a partial fix for #105
The extensions build locally without issues and seem to work well.