Update and lint discovery.azure to discovery.dockerswarm

[clayton-cornell]

PR Description

Splitting Discovery updates into multiple PRs to make reviews a bit easier.

Changes include:

• Pretty print tables
• Sort table rows alphabetically - required first, then all optional
• Sort blocks alphabetically
• Fix block heading hierarchy
• Add badge for Required blocks
• Remove block from heading text
• Remove extra spaces and extra CR/LFs
• Fix capitalization
• Tidy markdown - spaces around lists, indentation, etc.
• Reduce the passive voice

Which issue(s) this PR fixes

Notes to the Reviewer

PR Checklist

• CHANGELOG.md updated
• Documentation added
• Tests updated
• Config converters updated

[github-actions]

💻 Deploy preview deleted.

[clayton-cornell]

Related issue #2411

[clayton-cornell]

@dehaansa There's been quite a few changes since you last checked this. Might be worth a re-check.

I've noticed inconsistent use of double quotes in the Usage sections. I'm not sure if this is intentional or not.

For example in discovery.docker it is:

discovery.docker "<LABEL>" {
  host = <DOCKER_ENGINE_HOST>
}

and in discovery.dockerswarm it is:

discovery.dockerswarm "<LABEL>" {
  host = "<DOCKER_DAEMON_HOST>"
  role = "<SWARM_ROLE>"
}

[thampiotr]

I like the better use of space in the tables, thanks!

[thampiotr]

@dehaansa There's been quite a few changes since you last checked this. Might be worth a re-check.

I've noticed inconsistent use of double quotes in the Usage sections. I'm not sure if this is intentional or not.

I think the second example is preferrable - when working with strings, we should always use "" syntax.

[clayton-cornell]

when working with strings, we should always use "" syntax.

I'll do a pass through the Usage and Examples to check for this and update.