This repository was archived by the owner on Dec 6, 2024. It is now read-only.
prepare 2.9.1 release #78
Merged
Conversation
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
* bump version variable * update changelog
* add much more detailed build instructions * update PCRE url text * remove GMP from linux CI
* add pedantic flag back * add back wall and wextra * make more idiomatic
* adding a circleci badge to the readme * fix casing
Update c-sdk-common commit to 0b7ed7f25eeac75a3decacdf79a6041688fba954.
* Properly return EVAL_SCHEMA error when variation index out of bounds This changes the Ldi_evaluate return (and thus BoolVariation, etc) to return an evaluation error when the variation index is < 0 or otherwise out of bounds. Part of the problem here was an implicit cast from double to unsigned integer, which was makes the problem hard to spot in code. * Separate getting a value from adding a value. (#212) Co-authored-by: Ryan Lamb <4955475+kinyoklion@users.noreply.github.com>
…ring (#213) * Fix a couple more AllFlagsState contract tests (excluding experimentation test) * Refactor AllFlagsState tests to stop using internal interfaces, add comments.
* Fix null pointer causing segfault within event processor * Add unit test for use-after-free * Fix memory leak due to failure to clear LDDetails
* Refactor cmake project tests This commit adds a "test_" prefix to each cmake project test. Additionally, it factors out the boilerplate required to setup each test into a few cmake macros which can be used in a more declarative manner.
git-subtree-dir: c-sdk-common git-subtree-split: 0b7ed7f25eeac75a3decacdf79a6041688fba954
Re-add c-sdk-common subtree at same commit.
* Squashed 'c-sdk-common/' changes from 0b7ed7f2..2838fe5b 2838fe5b Make LDValue symbols visible (#25) fd393136 Add LDValue C++ bindings, use in tests (#23) ff6271bc Add valgrind memory leak checker to CI (#24) git-subtree-dir: c-sdk-common git-subtree-split: 2838fe5b2bfd7373a58009f79003ad147d1d1637 * Add cmake project test for C++ projects
Bumps the Linux docker image to ld-c-sdk-ubuntu:5, which added a necessary curl dependency that's used by aws-cli. Additionally, adds the empty CircleCI contexts to Linux, Mac, and Windows builds; also required by aws-cli.
These fixed feature event tests are a side-effect of PR #175 (292f12). Before that commit, the eval algorithm couldn't detect malformed flag (when variation index was out of bounds), so the event generation code didn't set the eval reason to MALFORMED_FLAG as would be expected.
* contract-tests: Only pass valid details pointer into evaluations when requested
* Fix debug event generation Before this commit, debug events were not queued up by the EventProcessor because of faulty logic. This commit modifies the processor to enqueue debug events as expected, and additionally makes changes to the JSON structure of debug events to make them conform to the specification.
Adds a new job in CI, which builds the SDK using a CircleCI M1 executor (arm64). This does not add any new release artifacts to the releaser configuration yet.
cwaldren-ld
approved these changes
May 5, 2023
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
[2.9.1] - 2023-05-05
Fixed: