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
vscode backports #52043
vscode backports #52043
Conversation
(cherry picked from commit a23e3b0)
(cherry picked from commit 874b017)
(cherry picked from commit d41e868)
(cherry picked from commit 38d4c09)
(cherry picked from commit 285800d)
(cherry picked from commit 4c95508)
(cherry picked from commit a53a3bf)
(cherry picked from commit f8ef8bf)
(cherry picked from commit c4cb982)
(cherry picked from commit 728d2a2)
(cherry picked from commit f7b05ca)
(cherry picked from commit 7369c3a)
(cherry picked from commit 70092fe)
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 the packages, but agree that we should backport vscode.
I went ahead in tested this in a vm with xfce with environment.systemPackages = [
(pkgs.vscode-with-extensions.override {
vscodeExtensions = with pkgs.vscode-extensions; [
bbenoist.Nix
ms-python.python
ms-vscode.cpptools
WakaTime.vscode-wakatime
];
})
]; Also without the extensions. @Mic92 Do you think we need a darwin tester? |
maybe @davidak could have a short look at the macOS version. |
@Mic92 i currently don't use macOS, sorry. maybe ask on #nix-darwin? |
Motivation for this change
Was reviewing #52036 and I recalled that we should be backporting this stuff to stable.
I've also included all changes to the vscode-extensions.
Let me know if I missed anything.
cc @Mic92 (since you brought this to our attention a bit back)
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)