Find a bug, or have an additional feature you'd like to propose? We welcome your contributions! Please read through the guidelines below and submit a Pull Request from your fork.
- Fork this repository to your GitHub user space
- Ensure your fork's master branch has the latest changes from this repository's master branch
- Clone your fork's master branch to your local file system
git clone <YOURFORK> cd mlctl
- Create a new local branch
pip install --editable .
This will install a version to an isolated environment in editable
mode. As you update the code in the repository, the new code will
immediately be available to run within the environment (without the
need to pip install
it again).
- Once your code changes are done, add tests here
- Run the tests using
tox
:pip install tox tox
mlctl
project will follow this branching structure:
develop
- Features that are actively being developed on. Most unstable branch, builds might failcanary
- As thedevelop
branch matures, more stable code can move to thecanary
branch. In this branch, features work together, builds are green and the overall code is POC quality. From this branch, code can either go tobeta
or tomain
.beta
- More polished POCs that maybe demo ready.main
- Release version of the library. Well tested and stable with a high bar for engineering quality.
- Push your local branch with your tested changes to your forked repo
- Open up a pull-request from your fork repo's branch to this repository's
develop
orcanary
branch, depending on the quality of the code- Include a brief summary of your changes
- Ensure all CICD branch checks pass
- Obtain at least one code review from a maintainer or trusted committer. See CODEOWNERS for details.
Before submitting an issue, please verify that an existing issue has not been reported already. Reported issues can be found at https://github.com/intuit/mlctl/issues.
When submitting an issue, please keep in mind to include the following:
- A detailed description of the bug
- Steps to reproduce the bug
- Version of
mlctl
and dependencies invovled