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

weather: Fix GetCurrent* functions returning future events #1879

Merged

Conversation

FintasticMan
Copy link
Member

It was potentially possible that an event with a timestamp in the future could be returned as a current event.

@FintasticMan FintasticMan added bug Something isn't working weather Bugs and PRs related to Weather labels Oct 6, 2023
@github-actions
Copy link

github-actions bot commented Oct 6, 2023

Build size and comparison to main:

Section Size Difference
text 376784B 0B
data 940B 0B
bss 63420B 0B

@FintasticMan FintasticMan merged commit eac460f into InfiniTimeOrg:main Oct 6, 2023
7 checks passed
@FintasticMan FintasticMan deleted the weather_fix_getcurrent_future branch October 6, 2023 17:54
Zetabite pushed a commit to Zetabite/InfiniTime that referenced this pull request Nov 12, 2023
@FintasticMan FintasticMan added this to the 1.14.0 milestone Dec 10, 2023
mark9064 pushed a commit to mark9064/InfiniTime that referenced this pull request Jan 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working weather Bugs and PRs related to Weather
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants