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
Bump VSCode version #98088
Bump VSCode version #98088
Conversation
Unfortunately this is blocked on VSCodium/vscodium#500 |
Should be closed? 1.50 is out. |
@srid Oh good point. I'll pump to 1.50 right away. (Or do you want a separate PR for that?) |
Unfortunately blocked on VSCodium/vscodium#529. The |
2fc70f2
to
e8c45fa
Compare
@srid This is now fixed and rebased. /marvin opt-in |
Hi! I'm an experimental bot. My goal is to guide this PR through its stages, hopefully ending with a merge. You can read up on the usage here. |
Reminder: Please review! This Pull Request is awaiting review. If you are the assigned reviewer, please have a look. Try to find another reviewer if necessary. If you can't, please say so. If the status is not accurate, please change it. If nothing happens, this PR will be put back in the |
@srid can you review or merge? |
Huh. upkeep-bot was faster. I wonder whether there is any point in doing this. |
@eadwu @Synthetica are vscode/vscodium versions bumped automatically when the script works fine? |
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'm not sure as I never used the update script, nor do I use the stable version.
Verified hashes.
Yup! You can see the trace of all previous runs of the bot here: https://github.com/samuela/nixpkgs-upkeep/actions. It creates PRs whenever the update scripts work. I've found that they only ever really break when vscode has a new release but the vscodium version is not yet released. |
That's great to know. I've documented this now, so people won't waste their time so much in the future. |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
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.
Could you please update the commit message for the version bump to follow the contribution guidelines?
vscode, vscodium: 1.50.0 -> 1.50.1
This may seem like nitpicking, but makes it easier to search or process the logs.
Also, the second commit message should be something like:
Feel free to ping me when these changes are made to get this merged. |
@danieldk Done :) |
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.
Thanks for updating the commit messages. Looks good to me now.
Result of nixpkgs-review pr 98088
1
3 packages built:
- vscode
- vscode-with-extensions
- vscodium
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)