Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: fix broken "Current status" badges #5783

Merged
merged 1 commit into from
Oct 10, 2023
Merged

docs: fix broken "Current status" badges #5783

merged 1 commit into from
Oct 10, 2023

Conversation

wojtekmaj
Copy link
Contributor

What's the problem this PR addresses?

Badge URLs were outdated (not in a format that "Create status badge" outputs function right now), causing them to be "stuck in the past". As a result, README shown both Angular E2E workflows as failing, where in fact they are running just fine!

How did you fix it?

Recreated all badges

Checklist

  • I have set the packages that need to be released for my changes to be effective.
  • I will check that all automated PR checks pass before the PR gets reviewed.

Badge URLs were outdated (not in a format that "Create status badge" outputs function right now), causing them to be "stuck in the past". As a result, README shown both Angular E2E workflows as failing, where in fact they are running just fine!
@arcanis
Copy link
Member

arcanis commented Oct 10, 2023

Good catch, I didn't realize GH broke those urls 😯

@arcanis arcanis merged commit d181aa6 into yarnpkg:master Oct 10, 2023
16 checks passed
@wojtekmaj wojtekmaj deleted the make-e2es-green-again-3 branch October 10, 2023 11:01
merceyz pushed a commit that referenced this pull request Oct 21, 2023
**What's the problem this PR addresses?**

Badge URLs were outdated (not in a format that "Create status badge"
outputs function right now), causing them to be "stuck in the past". As
a result, README shown both Angular E2E workflows as failing, where in
fact they are running just fine!

**How did you fix it?**

Recreated all badges

**Checklist**
<!--- Don't worry if you miss something, chores are automatically
tested. -->
<!--- This checklist exists to help you remember doing the chores when
you submit a PR. -->
<!--- Put an `x` in all the boxes that apply. -->
- [x] I have read the [Contributing
Guide](https://yarnpkg.com/advanced/contributing).

<!-- See
https://yarnpkg.com/advanced/contributing#preparing-your-pr-to-be-released
for more details. -->
<!-- Check with `yarn version check` and fix with `yarn version check
-i` -->
- [x] I have set the packages that need to be released for my changes to
be effective.

<!-- The "Testing chores" workflow validates that your PR follows our
guidelines. -->
<!-- If it doesn't pass, click on it to see details as to what your PR
might be missing. -->
- [x] I will check that all automated PR checks pass before the PR gets
reviewed.

(cherry picked from commit d181aa6)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants