Skip to content
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

Add vscode-insiders #327

Closed
wants to merge 1 commit into from
Closed

Add vscode-insiders #327

wants to merge 1 commit into from

Conversation

NatoBoram
Copy link

@NatoBoram NatoBoram commented May 19, 2020

Closes #320

Microsoft Reviewers: Open in CodeFlow

@wingetbot
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@wingetbot wingetbot added the Azure-Pipeline-Passed Validation pipeline passed. There may still be manual validation requirements. label May 19, 2020
@ghost ghost assigned KevinLaMS May 19, 2020
@ghost
Copy link

ghost commented May 19, 2020

This submission has moved to manual review.

Copy link
Member

@Tyriar Tyriar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I started a conversation about how to do this in https://github.com/microsoft/winget-pkgs/issues/278. The comments are for @KevinLaMS

@@ -0,0 +1,13 @@
Id: Microsoft.VisualStudioCodeInsiders
Version: 1.46.0
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is technically not correct as the version is x.y.z-insider, but having the version number would require us to change it once a month. I was wondering in https://github.com/microsoft/winget-pkgs/issues/278 if we can use something like "Latest" or "Insider" here instead.

Name: Visual Studio Code Insiders
Publisher: Microsoft Corporation
License: Copyright (c) Microsoft Corporation
AppMoniker: vscode-insiders
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I want to get #277 sorted out before we do insiders, ideally we would use code and code-insiders to match the CLI and have vscode as an alias/search term.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, I added my opinion on it!

Homepage: https://code.visualstudio.com/insiders/
Installers:
- Arch: x64
Url: https://go.microsoft.com/fwlink/?Linkid=852155
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This would probably be better: https://update.code.visualstudio.com/latest/win32-x64-user/insider, the hash won't work though as it changes every day. While certainly possible, it would be preferable not to need a PR (or automated commit) everyday to do this. I also saw that sources (winget source) are supported, are all Microsoft packages meant to be maintained in this repo? It would be easier to automate if we owned the repo.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Scoop also had to deal with this in ScoopInstaller/Scoop#181, and they added simplistic nightly support. Might be worth to see that they did.

@KevinLaMS
Copy link
Contributor

Duplicate of #2056
#2056
Would you be OK closing now that Vscode is owning updating this?

@KevinLaMS KevinLaMS added Needs: author feedback and removed Azure-Pipeline-Passed Validation pipeline passed. There may still be manual validation requirements. labels Jun 23, 2020
@NatoBoram
Copy link
Author

Sure!

@NatoBoram NatoBoram closed this Jun 23, 2020
@ghost ghost removed the Needs: author feedback label Jun 23, 2020
seejdev pushed a commit to seejdev/winget-pkgs that referenced this pull request Nov 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Request] Add VSCode Insiders
4 participants