Skip to content

Update health_check.yml#23176

Merged
KevinMind merged 2 commits intomasterfrom
kevinmind/addons/15456
Mar 17, 2025
Merged

Update health_check.yml#23176
KevinMind merged 2 commits intomasterfrom
kevinmind/addons/15456

Conversation

@KevinMind
Copy link
Copy Markdown
Contributor

Fixes: mozilla/addons#15456

Description

Fix the typo in the check for scheduled health check workflow runs.

Testing

Need a failure on prod with the scheduled run to verify this.. a bit hard to recreate.

Checklist

  • Add #ISSUENUM at the top of your PR to an existing open issue in the mozilla/addons repository.
  • Successfully verified the change locally.
  • The change is covered by automated tests, or otherwise indicated why doing so is unnecessary/impossible.
  • Add before and after screenshots (Only for changes that impact the UI).
  • Add or update relevant docs reflecting the changes made.

@KevinMind KevinMind requested a review from diox March 17, 2025 09:38
@KevinMind KevinMind merged commit 6d5eed8 into master Mar 17, 2025
42 checks passed
@KevinMind KevinMind deleted the kevinmind/addons/15456 branch March 17, 2025 10:25
KevinMind added a commit that referenced this pull request Mar 17, 2025
* Fix typo in healtcheck(_completed).yml files so the slack notifications will be sent on scheduled workflow runs.
KevinMind added a commit that referenced this pull request Mar 17, 2025
* Fix typo in healtcheck(_completed).yml files so the slack notifications will be sent on scheduled workflow runs.
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.

[Bug]: Healtcheck skipping slack notification when it should not

2 participants