Skip to content

Latest commit

 

History

History
144 lines (94 loc) · 4.48 KB

CONTRIBUTING.md

File metadata and controls

144 lines (94 loc) · 4.48 KB

Contributing

Contributions are welcome!

Types of contributions

Report bugs or submit feedback

Report bugs and submit feetback here.

Fix bugs

Look through the GitHub issues for bugs. Anything tagged with "bug" and "help wanted" is open to whoever wants to implement it.

Implement features

Look through the GitHub issues for features. Anything tagged with "enhancement" and "help wanted" is open to whoever wants to implement it.

Coding conventions

Netbox Style Guide
Django Coding style

For this:
All files will be formated using the black auto-formatter.
Configuration is stored in pyproject.toml

isort is used to automate import sorting.

Linting and PEP8 style enforcement will be done with Flake8 which is a wrapper arround:

  • PyFlakes
  • pycodestyle
  • Ned Batchelder’s McCabe script Configuration is maintained in the .flake8 file (no support for pyproject.toml)

The pre-commit Python framework is used to simplify the managment of pre-commit hooks.
Config is stored in .pre-commit-config.yaml

Repository structure

There are 2 permanent branch in the repository:

  • master - The current stable release. Individual changes should never be pushed directly to this branch, but rather merged from develop.
  • develop - Active development for the upcoming patch release. Pull requests will typically be based on this branch unless they introduce breaking changes that must be deferred until the next minor release.

For each new feature or bug fix a branch is created from the corresponding issue.

Setup your development environment

  1. Install Netbox. Make sure taht at the Netbox installtion step you follow the "Option B: Clone the Git Repository"

  2. From the Netbox directory you activate the NetBox virtual environment

    $ cd netbox
    $ source venv/netbox/bin/activate
    
  3. Fork the netbox-contract repo on GitHub.

  4. Clone your fork locally

    $ cd ..
    $ git clone [email protected]:your_name_here/netbox-contract.git
    
  5. Add the plugin to NetBox virtual environment:

    $ cd  netbox-contract
    $ python3 -m pip install -e .
    
  6. install pre-commit:

    python -m pip install pre-commit
    pre-commit install
  7. Update the Netbox configuration and run the database migrations as mentionned in the plugin installation steps.

  8. Create a branch for local development:

    $ git checkout -b name-of-your-bugfix-or-feature
    

    Make your changes locally.

  9. You can test your changes using the django development server:

    $ python3 netbox/netbox/manage.py runserver 0.0.0.0:8000 --insecure
    

    Connect to the name or IP of the server (as defined in ALLOWED_HOSTS) on port 8000; for example, http://127.0.0.1:8000/.

  10. Commit your changes and push your branch to GitHub:

    $ git add .
    $ git commit -m "Your detailed description of your changes."
    $ git push origin name-of-your-bugfix-or-feature
    
  11. Submit a pull request through the GitHub website.

Model changes

Create the migrations:

source netbox/venv/bin/activate
python3 netbox/netbox/manage.py makemigrations netbox_contract --dry-run
python3 netbox/netbox/manage.py makemigrations netbox_contract

Apply the migrations:

source netbox/venv/bin/activate
python3 netbox/netbox/manage.py migrate

Pull Request Guidelines

Before you submit a pull request, check that it meets these guidelines:

  1. The pull request should include tests.
  2. If the pull request adds functionality, the docs should be updated.
  3. The pull request should work for Python 3.8, 3.9, 3.10 and 3.11. Check Actions and make sure that the tests pass for all supported Python versions.

Deploying

A reminder for the maintainers on how to deploy. Make sure all your changes are committed (including an entry in CHANGELOG.md) and that all tests pass. Then in the github project go to Releases and create a new release with a new tag. This will automatically upload the release to pypi: