justinmayer / kagi

WebAuthn security keys and TOTP multi-factor authentication for Django
BSD 2-Clause "Simplified" License
91 stars 10 forks source link

Enhance README with various minor improvements #33

Closed justinmayer closed 5 years ago

justinmayer commented 5 years ago

And now that I think about it... I think we can remove the invoke setup task from README > Demo. Since Poetry is installed in a previous step, and since Pre-commit isn't needed to get the demo project running, I think perhaps this step is a bit superfluous.

I replaced setup with migrate via https://github.com/justinmayer/kagi/pull/33/commits/2a64a9276c120e3f73fa32b2890a9f0f3b2171f1, which we actually do need and was missing.

justinmayer commented 5 years ago

@Natim: Any comments on this PR before I merge it?