Clarify acceptable_conclusions format

[MattiSG]

Hi @CasperWA and thanks for push-protected! 🙂

We use this action as part of our workflow on @OpenTermsArchive and, when combining it with @OpenTermsArchive/changelog-action, we need to skip changelog checks on our main branches.

As part of setting up push-protected, I spontaneously read “list of acceptable statuses” in the acceptable_conclusions documentation as [ 'success', 'skipped' ]. This changeset suggests a more explicit way to document the expected format. I had a quick look at changing the parsing itself, to support lists or at least trim the entries in the string, but I understand that you use Python’s native in to check values, and changing this would be a bigger intervention on the codebase.

Cheers!
Matti

[CasperWA]

Cheers, this is indeed more clear, especially when including the example 👍

[CasperWA]

I will merge as is. The tests here will currently not run as intended due to limited access to GitHub secrets from external contributors, but the change should and will never have an effect on any of the current CI tests.