saltstack-formulas / template-formula

SaltStack formula template filled with dummy content
http://docs.saltstack.com/en/latest/topics/development/conventions/formulas.html
Other
119 stars 85 forks source link

README - have meta-states and sub-state headings. #169

Open noelmcloughlin opened 4 years ago

noelmcloughlin commented 4 years ago

I would recommend "Available States" section in README is split.

See example at https://github.com/saltstack-formulas/lvm-formula#available-meta-states

baby-gnu commented 4 years ago

I agree that we should provide a better way to distinguish between meta and sub states and separate components. Maybe using the order in which they are called?

For example:

Available Meta states

Setup

Clean

Available substates

Setup

Clean

baby-gnu commented 4 years ago

I think that if we have more than one configuration file, we should consider template.config as a meta-state and each individual file as sub-state, for example:

Available Meta states

Available substates

noelmcloughlin commented 4 years ago

Substates are really useful for advanced users needing granularity, or building a solution needing to call one substate (i.e.setup a repository) without forcing entire formula on them.

Since many software has official API the sub-states can match API calls. See lvm-formula and devstack-formula.cli states for example - the API = SUB-STATES.

Related to this is https://github.com/saltstack-formulas/template-formula/pull/136 since there is duplication of states when the README is generated.

myii commented 4 years ago

Available Meta states

...

Available substates

@baby-gnu While reviewing the lvm-formula PR, I'd like to suggest a minor adjustment to these headings:

-Available Meta states
+Available meta-states

...

-Available substates
+Available sub-states
baby-gnu commented 4 years ago

@myii I completely agree, this was just a proposal for discussion.

Thanks.

myii commented 4 years ago

@baby-gnu You're welcome, thanks for the feedback.