Update build status badge link URL

[jsf9k]

🗣 Description

This pull request updates the format of the build status badge URL that appears in the Mustache template.

💭 Motivation and context

I'm not sure what happens on the backend, but using the newer URL format seems to give more reliable results. Right now, for instance, the old URL format indicates failure for cisagov/skeleton-ansible-role-with-test-user while the new URL format correctly indicates success.

🧪 Testing

All automated tests pass.

✅ Pre-approval checklist

• This PR has an informative and human-readable title.
• Changes are limited to a single goal - eschew scope creep!
• All relevant type-of-change labels have been added.
• I have read the CONTRIBUTING document.
• These code changes follow cisagov code standards.
• All new and existing tests pass.
• Bump major, minor, patch, pre-release, and/or build versions as appropriate via the bump_version script if this repository is versioned and the changes in this PR warrant a version bump.

✅ Pre-merge checklist

• Finalize version.

✅ Post-merge checklist

• Create a release (necessary if and only if the version was bumped).

[dav3r]

👍

[mcdonnnj]

I think the new URL, which follows the documentation, uses the default branch unless overridden, whereas the old one is the workflow's last run's status without regard for branch. That would align with the results you're seeing since the last run on develop is a pass and the last run period is a failure.

[jsf9k]

I think the new URL, which follows the documentation, uses the default branch unless overridden, whereas the old one is the workflow's last run's status without regard for branch. That would align with the results you're seeing since the last run on develop is a pass and the last run period is a failure.

I think you're right. I also think the new URL aligns with the changes I made in cisagov/action-apb#69 and the intent of APB.