Add a plugins section to the README by maraino · Pull Request #1447 · smallstep/cli

Comment on lines +99 to +100

[`step-kms-proxy`] is also integrated directly into `step` to create
certificates, generate CSRs, sign tokens, and more using KMS-backed keys.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this go with the step-kms-plugin bullet? Or should it be step-kmsproxy-plugin?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried to do that but in markdown it goes to a new paragraph.

  • step-kms-plugin data ta data da long line .... long line ... long line ... long line ... long line ... long line ... long line ... long line

This plugin is also integrated

  • step-kmsproxy-plugin foo bar zar

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's in a new paragraph:
image

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But can't it be combined in the same bullet as step-kms-plugin, in a new sentence right after, by putting it on the same line? Or do you intentionally want this to be separate, because of the "integrated directly" being a more tight integration between step and the plugin? Even in that case, I think it'd be good to put it with the rest of the bullet, as this list could grow, and it being more out of context in that case.