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.

Sign up for GitHub

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

Jump to bottom

Task/flag sets #210

Merged
merged 28 commits into from
Nov 29, 2023
Merged

Task/flag sets #210

merged 28 commits into from
Nov 29, 2023

Conversation

nmayorsplit
Copy link
Contributor

GO SDK

What did you accomplish?

How do we test the changes introduced in this PR?

Extra Notes

nmayorsplit and others added 27 commits July 18, 2023 16:23
@nmayorsplit
Merge pull request #202 from splitio/development
4d3790c 
[SDKS-7122] Release IFF
@nmayorsplit
WIP:add treatmentByFlagSet
821e686
@nmayorsplit
[SDKS-7679] Implement getTreatments and create FlagSetsFilter in factory
6556fcc
@nmayorsplit
FlagSet warnings point to nil after print them
e65392a
@nmayorsplit
Pr suggestions
024222f
@nmayorsplit
Fix print flag sets warnings
b2cca49
@nmayorsplit
Add test case to validate flag set warnings
ed04468
@nmayorsplit
Pr suggestions
52cd6c6
@nmayorsplit
Merge pull request #203 from splitio/SDKS-7679
bee091f 
[SDKS-7679]
@nmayorsplit
[SDKS-7683] Set init config telemetry with FlagSetsTotal and FlagSets…
758be91 
…Invalid
@nmayorsplit
[SDKS-7685] Add sets in SplitView
92b7f52
@nmayorsplit
Merge pull request #204 from splitio/SDKS-7683
ee13c64 
[SDKS-7683] Set init config telemetry with FlagSetsTotal and FlagSetsInvalid
@nmayorsplit
Merge pull request #205 from splitio/SDKS-7685
590666f 
[SDKS-7685] Add sets in SplitView
@nmayorsplit
[SDKS-7633] Add default treatment in SplitView
e8e314e
@nmayorsplit
Merge pull request #206 from splitio/SDKS-7633
034bc58 
[SDKS-7633] Add default treatment in SplitView
@nmayorsplit
[SDKS-7625] Add feature flag name in ready warning log
457e396
@nmayorsplit
[SDKS-7625] Pr suggestion
55dd7c7
@nmayorsplit
Merge pull request #207 from splitio/SDKS-7625
7f9945d 
[SDKS-7625] Add feature flag name in ready warning log
@nmayorsplit
[SDKS-7687] Add test to validate flag sets when fetch ff
c428b46
@nmayorsplit
Set FlagSetsTotal and FlagSetsInvalid to 0 when consumer mode
b872c6b
@nmayorsplit
Merge pull request #208 from splitio/not-flag-set-filter
31cfef9 
Set FlagSetsTotal and FlagSetsInvalid to 0 when consumer mode
@nmayorsplit
After merge
eb2fa87
@nmayorsplit
[SDKS-7687] Add test case in input_validator
d308e41
@nmayorsplit
Merge pull request #209 from splitio/SDKS-7687
349818c 
[SDKS-7687] Add integration tests
@nmayorsplit
Udate common version
e9dbaef
@nmayorsplit
Update version for tapps
0d30b2c
@nmayorsplit
Update commons version and change logs
318d83b
@nmayorsplit nmayorsplit requested a review from a team as a code owner November 29, 2023 14:36
@sonarqube-pull-requests SonarQube Pull Requests
Copy link

SonarQube Quality Gate

Quality Gate failed

Failed condition 79.2% 79.2% Coverage on New Code (is less than 80%)

See analysis details on SonarQube

@nmayorsplit nmayorsplit merged commit 89ed66a into development Nov 29, 2023
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sanzmauro sanzmauro sanzmauro approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@nmayorsplit @sanzmauro