Skip to content

Conversation

RyanPrussin
Copy link
Contributor

@RyanPrussin RyanPrussin commented Jul 24, 2025

Overview

Remove warning when importing the 1Password/shell-plugins module in a nix flake.

evaluation warning: _1password has been renamed to _1password-cli to better follow upstream name usage

Type of change

  • Created a new plugin
  • Improved an existing plugin
  • Fixed a bug in an existing plugin
  • Improved contributor utilities or experience

Related Issue(s)

#527

How To Test

  • Manual Tests: in order to replicate the warning and confirm that it is fixed, you would have to import the package into a separate flake using home-manager, and then run nix flake check in that flake. Unfortunately I think you have no choice but to test it this way right now.
  • Automated Tests: you probably should be able to nix flake check in order to see these kind of warnings in your library directly, however this does not work with the current flake.nix configuration you have, and it would require more significant updates to your Nix code in order to support. Perhaps something worth considering.

Changelog

Nix Support: Remove evaluation warning when importing this module in a nix flake.

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.

1 participant