purescript / spago

🍝 PureScript package manager and build tool
BSD 3-Clause "New" or "Revised" License
792 stars 132 forks source link

Implement auth and transfer commands #1298

Open f-f opened 3 weeks ago

f-f commented 3 weeks ago

We fix #1123 by adding the auth and registry transfer commands, to allow for authenticated interactions with the Registry. The unpublish command is still unimplemented, but it should be easy now that the rest of the scaffolding is in place.

This PR includes a whole set of tests to verify that things do what they should, and docs to inform users how the authenticated flow looks like.

@fsoikin I intend to merge this before #1296 since I've been rebasing it for months already and I'd love to not have to do it once more 😄

(this fixes #1294 as well)

fsoikin commented 3 weeks ago

Absolutely no problem, I assumed it would be this way. I'm happy to keep rebasing until it's time.

fsoikin commented 3 weeks ago

I'm afraid it'll take me a few days to review though. This is quite large.

f-f commented 3 weeks ago

@fsoikin no worries! That'll give me some time to have a good look at #1296 🙂

f-f commented 3 weeks ago

Of course happy to have @thomashoneyman's review as well here if it doesn't take away time from other things 🙂