chore(release): prepare v0.3.0 - #7
Merged
Merged
Conversation
A minor rather than a patch: the two-argument parse_fire_weather() overload is gone, because a version that silently guessed the layer was how the dn label confusion survived v0.2.0, and a compile error with a one-line fix is the honest way to surface that. The release carries two defects that were reachable by any consumer of the public rate limiter, neither of them through this SDK's own clients: a zero refill interval divided by zero, which traps on x86-64 and returns a silent wrong answer on AArch64, and a daily limit with no max_wait made acquire() spin until the next UTC midnight. It also stops HttpClient reaching the local filesystem: a file:// URL was classified as relative, appended to the empty default base URL, and handed to libcurl, which read the file and returned it as the response body.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Promotes the Unreleased section to 0.3.0 and bumps the version in the three places that carry it: the CMake project field the release workflow verifies the tag against, the README FetchContent example, and the
SPC_VERSION_STRINGfallback inhttp_client.hpp. No other 0.2.0 reference remains outside changelog history.Minor rather than patch because the two-argument
parse_fire_weather()overload was removed. A version that silently assumed the Outlook layer is how the dn label confusion survived v0.2.0, and the captured Day 1 and Day 2 fixtures carry no LABEL at all, so the layer argument is the only thing that disambiguates them.Release build and 65 tests pass, clang-format is clean, markdownlint reports zero errors.