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.

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

feat(python of provider): Stop using provider.provider #1772

Merged
merged 7 commits into from
Apr 30, 2024

Conversation

thomaspoignant
Copy link
Owner

Description

Stop using provider.provider as mentioned in open-feature/python-sdk#319

Next steps:

  • When the python SDK is available, bump the SDK version in poetry
  • Release a new version.

Checklist

Copy link

netlify bot commented Apr 19, 2024

Deploy Preview for go-feature-flag-doc-preview canceled.

Name Link
🔨 Latest commit 4357389
🔍 Latest deploy log https://app.netlify.com/sites/go-feature-flag-doc-preview/deploys/662220576aec8000081a8260

Copy link

netlify bot commented Apr 19, 2024

Deploy Preview for go-feature-flag-doc-preview canceled.

Name Link
🔨 Latest commit 91ec783
🔍 Latest deploy log https://app.netlify.com/sites/go-feature-flag-doc-preview/deploys/6630af235626ce00082fed64

Copy link

codecov bot commented Apr 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.75%. Comparing base (22882a1) to head (91ec783).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1772   +/-   ##
=======================================
  Coverage   86.75%   86.75%           
=======================================
  Files          89       89           
  Lines        3344     3344           
=======================================
  Hits         2901     2901           
  Misses        343      343           
  Partials      100      100           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@thomaspoignant
Copy link
Owner Author

Waiting for the SDK to be released and we will release this PR when available.

@@ -5,4 +5,7 @@

@dataclass
class GoFeatureFlagMetadata(Metadata):
def __init__(self):
pass

Choose a reason for hiding this comment

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

note: this is probably not necessary

Copy link
Owner Author

Choose a reason for hiding this comment

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

I've added it because pycharm was complaining 😮‍💨

@federicobond
Copy link

FYI provider state should be handled by the Python SDK according to the latest OF spec. There is no longer a need to expose a get_status method yourself.

Signed-off-by: Thomas Poignant <thomas.poignant@gofeatureflag.org>
@thomaspoignant
Copy link
Owner Author

FYI provider state should be handled by the Python SDK according to the latest OF spec. There is no longer a need to expose a get_status method yourself.

Thanks for the heads up @federicobond, I've removed the usage of get_status from the code.

Signed-off-by: Thomas Poignant <thomas.poignant@gofeatureflag.org>
@thomaspoignant thomaspoignant changed the title [python of provider] Stop using provider.provider feat(python of provider) Stop using provider.provider Apr 29, 2024
@thomaspoignant thomaspoignant changed the title feat(python of provider) Stop using provider.provider feat(python of provider): Stop using provider.provider Apr 29, 2024
Copy link

sonarcloud bot commented Apr 29, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@thomaspoignant thomaspoignant merged commit 55a85c9 into main Apr 30, 2024
21 checks passed
@thomaspoignant thomaspoignant deleted the python-provider-0.7.0 branch April 30, 2024 10:42
thomaspoignant added a commit that referenced this pull request Sep 5, 2024
* Stop using provider.provider as mentionned in open-feature/python-sdk#319

Signed-off-by: Thomas Poignant <thomas.poignant@gofeatureflag.org>

* Remove usage of status

Signed-off-by: Thomas Poignant <thomas.poignant@gofeatureflag.org>

* remove unused init

Signed-off-by: Thomas Poignant <thomas.poignant@gofeatureflag.org>

---------

Signed-off-by: Thomas Poignant <thomas.poignant@gofeatureflag.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants