shell-plugins icon indicating copy to clipboard operation
shell-plugins copied to clipboard

Add plugin for Nimble package manager

Open electrocally opened this issue 2 years ago • 5 comments

Overview

This plugin adds support for https://nimble.directory, the package manager for https://nim-lang.org

Type of change

  • [x] Created a new plugin

How To Test

  1. Install Nim, this will also install nimble
  2. Install this plugin using these instructions with make nimble/build & op plugin init nimble. This may prompt for your PAT, explained in step 4
  3. Run nimble init to create a new Nim package (additional docs here)
  4. When executing nimble publish, 1Password will prompt for your GitHub Personal Access Token. This will need public_repo permissions.

Changelog

Authenticate the publishing Nim (Nimble) packages using Touch ID and other unlock options with 1Password Shell Plugins.

electrocally avatar May 01 '23 09:05 electrocally

@electrocally Are there any further blockers you would need some help with, following our developer Slack conversation? Let me know 😄 !

AndyTitu avatar May 19 '23 11:05 AndyTitu

@electrocally Are there any further blockers you would need some help with, following our developer Slack conversation? Let me know 😄 !

Thanks for checking in @AndyTitu! I've not had a lot of time for my projects lately. Hoping to do a bit more on this over the coming evenings 😁

electrocally avatar May 22 '23 08:05 electrocally

@electrocally Do let us know if there is anything that we can help with, to keep this moving 😄

hculea avatar May 30 '23 08:05 hculea

Hi guys! Thanks so much for the continued encouragement the last few weeks. It's been really great to contribute to this project 😀

I've just pushed up some changes - there's good and bad news. The good: it works! I've tidied up some of the code but as always would love to hear any further suggestions/optimisations.

The not so good: I love the concept of using the Plugin: github setup & I've really appreciated all your help with it. It works brilliantly to provision credentials and can be reused across all sorts of apps that rely on GitHub. That said, as we discussed on Slack, until we can add in an Importer argument to the CredentialUsage type, I can't seem to find a way to get that configuration working with importer(s). For now I'd like to prioritise the onboarding process of the plugin to anyone using the default Nimble setup & would be more than happy to contribute to adding that functionality to the type definition

electrocally avatar Jun 06 '23 22:06 electrocally

@electrocally I'm fine with not attempting to change the schema to adhere to the importing scenario in this MR. Let's make sure we create a separate issue for that though, if one doesn't exist already.

But there are still a few open discussions to be resolved before we could merge this PR

AndyTitu avatar Aug 30 '23 10:08 AndyTitu