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
fix: Flutter client should depend only on common client. #113
Conversation
@@ -14,8 +14,7 @@ dependencies: | |||
sdk: flutter | |||
package_info_plus: ^5.0.1 | |||
device_info_plus: ^9.1.1 | |||
launchdarkly_common_client: 0.1.0 | |||
launchdarkly_dart_common: 0.0.2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removing this.
@@ -14,8 +14,7 @@ dependencies: | |||
sdk: flutter | |||
package_info_plus: ^5.0.1 | |||
device_info_plus: ^9.1.1 | |||
launchdarkly_common_client: 0.1.0 | |||
launchdarkly_dart_common: 0.0.2 | |||
launchdarkly_common_client: 1.0.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will need updated again after the publish.
…arkly/flutter-client-sdk into rlamb/fix-dependency-propagation
@@ -3,16 +3,13 @@ | |||
"separate-pull-requests": true, | |||
"packages": { | |||
"packages/common": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All, the 1.0.0 have been released, so now those should auto-bump from now on.
🤖 I have created a release *beep* *boop* --- ## [1.0.1](launchdarkly_common_client-v1.0.0...launchdarkly_common_client-v1.0.1) (2024-01-30) ### Bug Fixes * Flutter client should depend only on common client. ([#113](#113)) ([edfd06d](edfd06d)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
🤖 I have created a release *beep* *boop* --- ## [4.0.0](4.0.0-alpha.1...4.0.0) (2024-01-31) This version of the SDK has been re-written in dart and now supports Android, iOS, Windows, macOS, Linux, and Web. The previous versions were wrappers which utilized our native Android and iOS SDKs. A migration guide will be available in our [docs](https://docs.launchdarkly.com/sdk/client-side/flutter). ### Features * Add example project. ([#93](#93)) ([588ae01](588ae01)) * Implement support for REPORT for streaming. ([#96](#96)) ([1de6079](1de6079)) * Use version 1.0.1 of common client. ([#115](#115)) ([cd85c65](cd85c65)) ### Bug Fixes * Check for network state on resume. ([#95](#95)) ([c1cb489](c1cb489)) * Correct timeout handling in example. ([#97](#97)) ([dc18529](dc18529)) * Flutter client should depend only on common client. ([#113](#113)) ([edfd06d](edfd06d)) * Mac entitlements. Clarifications. ([#107](#107)) ([077e9aa](077e9aa)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). --------- Co-authored-by: Ryan Lamb <4955475+kinyoklion@users.noreply.github.com>
I noticed the flutter SDK directly depended on common. This was because common client was missing some re-exports. This fixes that. So I will release the patch of common client after merging this. Then do the final version number updates.