Skip to content

Flag sets#100

Merged
emmaz90 merged 12 commits into
developmentfrom
sdks-7437
Nov 3, 2023
Merged

Flag sets#100
emmaz90 merged 12 commits into
developmentfrom
sdks-7437

Conversation

@emmaz90

@emmaz90 emmaz90 commented Oct 26, 2023

Copy link
Copy Markdown
Contributor

JS Browser SDK

What did you accomplish?

How do we test the changes introduced in this PR?

Extra Notes

@emmaz90 emmaz90 requested a review from a team as a code owner October 26, 2023 21:13
Comment thread .github/workflows/ci-cd.yml Outdated

- name: Store assets
if: ${{ github.event_name == 'push' && (github.ref == 'refs/heads/development' || github.ref == 'refs/heads/main') }}
if: ${{ github.event_name == 'push' && (github.ref == 'refs/heads/sdks-7668' || github.ref == 'refs/heads/main') }}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sdks-7668 -> development

Comment thread types/splitio.d.ts
* @typedef {string} SplitFilterType
*/
type SplitFilterType = 'byName' | 'byPrefix';
type SplitFilterType = 'bySet' | 'byName' | 'byPrefix';

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

missing the sets property in SplitView type.
Check splitio/react-native-client#71

@sonarqube-pull-requests

Copy link
Copy Markdown

SonarQube Quality Gate

Quality Gate failed

Failed condition 0.0% 0.0% Coverage on New Code (is less than 80%)
Failed condition 6.8% 6.76% Duplicated Lines (%) on New Code (is greater than 3%)

See analysis details on SonarQube

@emmaz90 emmaz90 merged commit 074d1ad into development Nov 3, 2023
@emmaz90 emmaz90 deleted the sdks-7437 branch November 3, 2023 22:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants