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

vscode: improve update_installed_exts.sh #192849

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sheldonneuberger-sc
Copy link
Contributor

@sheldonneuberger-sc sheldonneuberger-sc commented Sep 25, 2022

  • Redirect stderr, because some vscode wrappers (like nixpkgs' vscodeEnv) will print info that breaks this script.
  • Add EXCLUDE param to exclude some extensions from being updated, which is useful for installed extensions that aren't in the marketplace.

I tested both of these cases with my own use-case.

Description of changes
Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 22.11 Release Notes (or backporting 22.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
    • (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes
  • Fits CONTRIBUTING.md.

- Redirect stderr, because some vscode wrappers (like nixpkgs'
  vscodeEnv) will print info that breaks this script.
- Add EXCLUDE param to exclude some extensions from being updated, which
  is useful for installed extensions that aren't in the marketplace.
@ofborg ofborg bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux labels Sep 25, 2022
@sheldonneuberger-sc
Copy link
Contributor Author

@jonringer, please review if you get a chance

@sheldonneuberger-sc
Copy link
Contributor Author

Still looking for a reviewer, maybe @bobby285271 ?

@sheldonneuberger-sc
Copy link
Contributor Author

@SuperSandro2000 I've tried a couple people but can't find a reviewer for this, can you help?

Comment on lines +6 to +10
# Optional EXCLUDE env var can be set to a string with a space-separated list
# of extensions to exclude from the output, e.g.:
# $ EXCLUDE="pub1.foo pub2.bar" ./update_installed_exts.sh
#
# Can be added to your configuration with the following command and snippet:
Copy link
Member

Choose a reason for hiding this comment

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

If this is piped anyway, wouldn't it be easier to just use grep to filter?

@wegank wegank added the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Jul 4, 2024
@FliegendeWurst FliegendeWurst added the awaiting_changes (old Marvin label, do not use) label Dec 1, 2024
@stale stale bot removed the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Dec 1, 2024
@wegank wegank added the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Jan 1, 2025
@SigmaSquadron SigmaSquadron removed the awaiting_changes (old Marvin label, do not use) label Jan 5, 2025
@stale stale bot removed the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Jan 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.topic: vscode 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants