3.9 KiB
Contributing
Development
System Requirements
Python 3.9 and above are required.
Target version(s)
Python 3.9 and above are supported by the SDK.
Installation and Dependencies
We use uv for fast Python package management and dependency resolution.
To install uv, follow the installation guide.
Setup Development Environment
-
Clone the repository:
git clone https://github.com/open-feature/python-sdk.git cd python-sdk -
Install dependencies:
uv sync --frozen
Testing
Run tests:
uv run test --frozen
Coverage
Run tests with a coverage report:
uv run cov --frozen
End-to-End Tests
Run e2e tests with behave:
uv run e2e --frozen
Pre-commit
Run pre-commit hooks
uv run precommit --frozen
Integration tests
These are planned once the SDK has been stabilized and a Flagd provider implemented. At that point, we will utilize the gherkin integration tests to validate against a live, seeded Flagd instance.
Packaging
We publish to the PyPI repository, where you can find this package at openfeature-sdk.
Pull Request
All contributions to the OpenFeature project are welcome via GitHub pull requests.
To create a new PR, you will need to first fork the GitHub repository and clone upstream.
git clone https://github.com/open-feature/python-sdk.git openfeature-python-sdk
Navigate to the repository folder
cd openfeature-python-sdk
Add your fork as an origin
git remote add fork https://github.com/YOUR_GITHUB_USERNAME/python-sdk.git
Ensure your development environment is all set up by building and testing
uv run test --frozen
To start working on a new feature or bugfix, create a new branch and start working on it.
git checkout -b feat/NAME_OF_FEATURE
# Make your changes
git commit
git push fork feat/NAME_OF_FEATURE
Open a pull request against the main python-sdk repository.
How to Receive Comments
- If the PR is not ready for review, please mark it as
draft. - Make sure all required CI checks are clear.
- Submit small, focused PRs addressing a single concern/issue.
- Make sure the PR title reflects the contribution.
- Write a summary that explains the change.
- Include usage examples in the summary, where applicable.
How to Get PRs Merged
A PR is considered to be ready to merge when:
- Major feedback is resolved.
- Urgent fix can take exception as long as it has been actively communicated.
Any Maintainer can merge the PR once it is ready to merge. Note, that some PRs may not be merged immediately if the repo is in the process of a release and the maintainers decided to defer the PR to the next release train.
If a PR has been stuck (e.g. there are lots of debates and people couldn't agree on each other), the owner should try to get people aligned by:
- Consolidating the perspectives and putting a summary in the PR. It is recommended to add a link into the PR description, which points to a comment with a summary in the PR conversation.
- Tagging domain experts (by looking at the change history) in the PR asking for suggestion.
- Reaching out to more people on the CNCF OpenFeature Slack channel.
- Stepping back to see if it makes sense to narrow down the scope of the PR or split it up.
- If none of the above worked and the PR has been stuck for more than 2 weeks, the owner should bring it to the OpenFeatures meeting.
Design Choices
As with other OpenFeature SDKs, python-sdk follows the openfeature-specification.