nushellPlugins.desktop_notifications: init at 0.106.1

[timon-schelling]

https://github.com/FMotalleb/nu_plugin_desktop_notifications

closes #374678

Things done

• Built on platform(s)
  • x86_64-linux
  • aarch64-linux
  • x86_64-darwin
  • aarch64-darwin
• For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
  • sandbox = relaxed
  • sandbox = true
• Tested, as applicable:
  • NixOS test(s) (look inside nixos/tests)
  • and/or package tests
  • or, for functions and "core" functionality, tests in lib/tests or pkgs/test
  • made sure NixOS tests are linked to the relevant packages
• 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/)
• Nixpkgs 25.11 Release Notes (or backporting 24.11 and 25.05 Nixpkgs Release notes)
  • (Package updates) Added a release notes entry if the change is major or breaking
• NixOS 25.11 Release Notes (or backporting 24.11 and 25.05 NixOS Release notes)
  • (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
• Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other contributing documentation in corresponding paths.

---

Add a 👍 reaction to pull requests you find important.

[nixos-discourse]

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/prs-ready-for-review/3032/5611

[timon-schelling]

nixpkgs-review result

Generated using nixpkgs-review-gha

Command: nixpkgs-review pr 419914

Logs: https://github.com/timon-schelling/run-nixpkgs-review/actions/runs/16022615146

---

x86_64-linux (sandbox = true)

✅ 1 package built:

• nushellPlugins.desktop_notifications

---

aarch64-linux (sandbox = true)

✅ 1 package built:

• nushellPlugins.desktop_notifications

[timon-schelling]

@NixOS/nixpkgs-vet

I think https://github.com/NixOS/nixpkgs/actions/runs/16022597093/job/45202899627?pr=419914 failed for a reason not related to this pr.

[wolfgangwalther]

Correct, this is easily fixed by re-running the job (which I just triggered). We're aware of that failure mode already (#413239), but haven't been able to fix it, yet.

[nixos-discourse]

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/prs-ready-for-review/3032/5728

[philiptaron]

this uses an underscore and the PR title uses a dash and the filename uses a dash. which is right?

[timon-schelling]

Changed PR title. @dtomvan do you know if nu plugins follow a convention for package and file name? Do you want me to use - or _ for both?

[dtomvan]

Nothing specific to nushell plugins AFAICT.

Relevant (general) package naming conventions snippets:

New attribute names should be the same as the value in pname.

Hyphenated names should not be converted to snake case or camel case. This was done historically, but is not necessary any more. The Nix language allows dashes in identifiers since 2012.

It should be identical to the upstream package name.

Given that the upstream name is nu_plugin_desktop_notifications I think the attrname nushellPlugins.desktop_notifications makes most sense, as pname also already reflects the upstream name with underscores too.

The file name should then probably also use underscores.

[philiptaron]

New release at 0.106.1 but otherwise LGTM

[timon-schelling]

nixpkgs-review result

Generated using nixpkgs-review-gha

Command: nixpkgs-review pr 419914

Logs: https://github.com/timon-schelling/run-nixpkgs-review/actions/runs/16698986600

---

x86_64-linux (sandbox = true)

✅ 1 package built:

• nushellPlugins.desktop_notifications

---

aarch64-linux (sandbox = true)

✅ 1 package built:

• nushellPlugins.desktop_notifications

[nixos-discourse]

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/prs-already-reviewed/2617/2527