Skip to content
New issue

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

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

Already on GitHub? # to your account

fix: treat temporary failures as activity #593

Merged
merged 1 commit into from
Nov 19, 2024
Merged

Conversation

languitar
Copy link
Owner

When an activity check fails with a TemporaryCheckError, we now treat that situation as activity to prevent unintentional shutdowns of the system due to having a check configured wrongly or simply because of a temporary name resolution error or alike.

Fixes: #589

Copy link

codecov bot commented Nov 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.79%. Comparing base (9a94515) to head (8c96853).
Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #593   +/-   ##
=======================================
  Coverage   95.79%   95.79%           
=======================================
  Files          20       20           
  Lines        1331     1331           
  Branches      115      115           
=======================================
  Hits         1275     1275           
  Misses         46       46           
  Partials       10       10           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.


🚨 Try these New Features:

@languitar languitar force-pushed the feature/erros-as-activity branch from 3cec889 to 59e2d54 Compare November 19, 2024 22:46
When an activity check fails with a TemporaryCheckError, we now treat
that situation as activity to prevent unintentional shutdowns of the
system due to having a check configured wrongly or simply because of a
temporary name resolution error or alike.

Fixes: #589
@languitar languitar force-pushed the feature/erros-as-activity branch from 59e2d54 to 8c96853 Compare November 19, 2024 22:46
@languitar languitar merged commit 6b94a71 into main Nov 19, 2024
12 checks passed
@languitar languitar deleted the feature/erros-as-activity branch November 19, 2024 22:48
@renovate-releases
Copy link

🎉 This PR is included in version 7.0.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

When a check errors, it should register as activity
1 participant