ensure-alphabetize should add a commit with alphabetized data if needed

[mlinksva]

That would be nice.

It's easy to cause the test to fail if one forgets how - sorts in a word or even adds superfluous space characters.

[mlinksva]

Some actions that may be useful, most stars first:

• https://github.com/marketplace/actions/add-commit
• https://github.com/marketplace/actions/git-auto-commit
• https://github.com/marketplace/actions/add-commit
• https://github.com/marketplace/actions/git-commit-and-push
• https://github.com/marketplace/actions/github-commit-push

[WebSnke]

@mlinksva What exactly would resolve this problem? Changing the entry itself would break the link, right?

[WebSnke]

I'm going to leave open PRs that are failing the alphabetization test (which also can fail due to extra spaces, like this one) as motivation to fix #991 but feel free to make pass manually and I'll merge when you do.

The tests seem to fail when new countries are added...

[mlinksva]

The tests sometimes fail when new countries are added because extra spaces are present, probably when adding a new line the via the web editor which indents to the previous level.

government.github.com/.github/workflows/ensure-alphabetize.yml

Lines 28 to 31 in 95b66b0

	- name: Ensure Orgs are sorted
	run: |
	bundle exec script/alphabetize
	git diff --color --exit-code

The above sorts and overwrites the file, as a side effect without any extra spaces. It fails if diff has any output, whether from extra spaces or need to sort.

The idea would be to commit the overwritten file if there is any diff, and push that commit, autocorrecting a file that differs from the canonical (including sorted) output rather than failing and requiring manual fixup.

[mlinksva]

Hopefully just like this example https://github.com/marketplace/actions/add-commit#automated-linting

[mlinksva]

Nearly working in #1011 (comment)

[WebSnke]

The tests pass now in #1043.