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

Fix monthly checks for UpdaterPlugin #1065

Merged

Conversation

dalyIsaac
Copy link
Owner

Previously, using a monthly check for updates would cause an integer overflow.

This was because of the maximum dueDate in the Timer class. This has been resolved by triggering the event daily and verify whether a check can occur on that date.

@dalyIsaac dalyIsaac added bug Something isn't working updater Whim.Updater labels Nov 2, 2024
@dalyIsaac dalyIsaac force-pushed the fix-updater-max-interval branch from 36efa50 to 5dd4c0a Compare November 2, 2024 07:19
Copy link

codecov bot commented Nov 2, 2024

Codecov Report

Attention: Patch coverage is 7.69231% with 12 lines in your changes missing coverage. Please review.

Please upload report for BASE (1063-provide-yaml-config-by-default@3b91392). Learn more about missing BASE report.

Files with missing lines Patch % Lines
src/Whim.Updater/UpdaterPlugin.cs 7.69% 12 Missing ⚠️
Additional details and impacted files
@@                          Coverage Diff                           @@
##             1063-provide-yaml-config-by-default    #1065   +/-   ##
======================================================================
  Coverage                                       ?   80.56%           
======================================================================
  Files                                          ?      283           
  Lines                                          ?    12408           
  Branches                                       ?     1432           
======================================================================
  Hits                                           ?     9997           
  Misses                                         ?     2226           
  Partials                                       ?      185           

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

@dalyIsaac dalyIsaac marked this pull request as ready for review November 2, 2024 07:36
@dalyIsaac dalyIsaac merged commit 2fadf98 into 1063-provide-yaml-config-by-default Nov 2, 2024
8 checks passed
@dalyIsaac dalyIsaac deleted the fix-updater-max-interval branch November 2, 2024 07:37
@dalyIsaac dalyIsaac restored the fix-updater-max-interval branch November 2, 2024 07:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working updater Whim.Updater
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant