saltstack-formulas / salt-formula

Yes, Salt can Salt itself!
http://docs.saltstack.com/en/latest/topics/development/conventions/formulas.html
Other
197 stars 423 forks source link

fix: sort extra options in configuration #518

Closed ze42 closed 2 years ago

ze42 commented 2 years ago

Ensure extra options in generated configuration is sorted, to ensure stable configuration.

PR progress checklist (to be filled in by reviewers)


What type of PR is this?

Primary type

Secondary type

Does this PR introduce a BREAKING CHANGE?

No.

Related issues and/or pull requests

Describe the changes you're proposing

Just ensure extra (not otherwised known) options are placed in a stable order.

Pillar / config required to test the proposed changes

Debug log showing how the proposed changes work

Documentation checklist

Testing checklist

Additional context

aboe76 commented 2 years ago

@myii would you like to do the honors?

myii commented 2 years ago

@myii would you like to do the honors?

@aboe76 No need to wait for me, please go ahead when you're happy with a PR. I haven't had much time to review PRs recently, so I'd appreciate others doing so.

saltstack-formulas-travis commented 2 years ago

:tada: This PR is included in version 1.9.9 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket: