Skip to content

[FSSDK-9096] chore: Prepare release 5.0.0-beta #822

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

Merged
merged 2 commits into from
May 3, 2023

Conversation

opti-jnguyen
Copy link
Contributor

Summary

  • Incremented Optimizely SDK version from 4.9.3 to 5.0.0-beta.1

Prepare for first v5 Beta Pre-release.

Test plan

  • All existing tests should pass.
  • New tests introduced in this version should also pass.
  • The known FSC failure will be addressed in a future patch.

Issues

@opti-jnguyen opti-jnguyen marked this pull request as ready for review May 3, 2023 21:55
@opti-jnguyen opti-jnguyen requested a review from a team as a code owner May 3, 2023 21:55
Copy link
Contributor

@jaeopt jaeopt left a comment

Choose a reason for hiding this comment

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

Can we rename to 5.0.0-beta to be consistent with others?
I expect that this PR will pass all FSC tests, correct?

@@ -7,7 +7,49 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.

## [Unreleased]

### Breaking Changes
## 5.0.0-beta.1
Copy link
Contributor

Choose a reason for hiding this comment

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

5.0.0-beta to be consistent with other sdks?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated to be consistent. This PR should pass all FSC tests minus two which involve firing queued events upon config change.

Upon further investigation, unsure if the FSC test should apply to the Browser variant as events are always immediately sent in a fire-and-forget method due to use of the Pixel API. Either way, will be publishing the Beta version as is, noting down all issues such as those, and addressing them in future patches.

@coveralls
Copy link

coveralls commented May 3, 2023

Coverage Status

Coverage: 90.557%. Remained the same when pulling f158ef2 on jnguyen/prepare-5.0.0-beta into 082c086 on master.

Copy link
Contributor

@jaeopt jaeopt left a comment

Choose a reason for hiding this comment

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

LGTM

@opti-jnguyen opti-jnguyen changed the title [FSSDK-9096] chore: Prepare release 5.0.0-beta.1 [FSSDK-9096] chore: Prepare release 5.0.0-beta May 3, 2023
@opti-jnguyen opti-jnguyen merged commit d11c847 into master May 3, 2023
@opti-jnguyen opti-jnguyen deleted the jnguyen/prepare-5.0.0-beta branch May 3, 2023 22:55
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