Skip to content

Minior changes to shell code#1

Merged
Ericson2314 merged 1 commit intoEricson2314:bsd-crossfrom
9glenda:bsd-cross
Oct 31, 2022
Merged

Minior changes to shell code#1
Ericson2314 merged 1 commit intoEricson2314:bsd-crossfrom
9glenda:bsd-cross

Conversation

@9glenda
Copy link
Copy Markdown

@9glenda 9glenda commented Oct 27, 2022

Motivation for this change

Cleaner code

Things done

Mainly added quotes where they are generally recommended to use

  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.

Hopefully I didn't break any build steps which I am pretty sure I didn't but I wasn't able to figure out how to test rather my code still builds.

@Ericson2314
Copy link
Copy Markdown
Owner

Thank you!

@Ericson2314 Ericson2314 merged this pull request into Ericson2314:bsd-cross Oct 31, 2022
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.

2 participants