Skip to content
This repository was archived by the owner on Oct 15, 2025. It is now read-only.

Conversation

@marcocen
Copy link

This adds an additional action to copy OTP using the pass-otp extension.

The check for the extension isn't pretty but the otp extension doesn't add a binary to $PATH, just a file to /usr/local/lib/password-store/extensions (if installed with the source Makefile) o /usr/lib/password-store/extensions (as installed by some distributions).

@shark0der
Copy link

@ManuelSchneid3r any reason not to merge this? I've been applying this manually after each albert update 😢

@ManuelSchneid3r
Copy link
Member

The reason is nobody cares to review Pull requests and i dont have that much time.

@ManuelSchneid3r
Copy link
Member

Look a simple information like yours is a pretty basic test which I dont have to do in the end. I cant just randmoly merge things. Especially if i personally dont use the extensions.

@ManuelSchneid3r
Copy link
Member

@marcocen please increase the version and put yourself in the authors list.

@ManuelSchneid3r
Copy link
Member

0.18 is out. Please check the new api . Do you mind to volunteer as a maintainer for the plugin?

@ManuelSchneid3r
Copy link
Member

@marcocen are you still interested in getting this plugin upstream?

@ManuelSchneid3r
Copy link
Member

I am going to close this PR. It is not that I dont want this to be shipped upstream, but, as is, it cant be merged due to conflicts. Users/devs get the impression that I dont want to merge plugins, probably simply due to the amount of open PRs. Also it's cumbersome for me to repeatedly iterate all these abandoned and stale PRs. Note that I am still interested in merging an up to date version of this plugin and I still get notifications. So if you want something to happen leave a comment or simply open a new PR.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants