Skip to content

Latest commit

 

History

History
45 lines (34 loc) · 2.08 KB

CONTRIBUTING.md

File metadata and controls

45 lines (34 loc) · 2.08 KB

Contributing to gurobipy-pandas

Welcome to gurobipy-pandas!

We value your experience in using gurobipy-pandas and would like to encourage you to contribute directly to this gurobipy-pandas.

How to report bugs or submit feature requests

If you encounter a bug, or you think there is a need for a new feature, we recommend to first add the bug report or feature request to the gurobipy-pandas's GitHub issue tracker.

It would be great if you add a minimal reproducible example when reporting a bug, or include reasoning on how the new requested feature improves the code.

Submitting changes

We welcome your contribution in directly tackling some of the issues.

We use the GitHub pull request workflow. Once your pull request is ready for review, one of the core maintainers of gurobipy-pandas will review your pull request.

A pull request should contain tests for the changes made to the code behavior, should include a clear message outlining the changes done, and should be linked to an existing issue.

Before submitting a pull request:

  • install the pre-commit package to enable the automatic running of the pre-commit hooks in the .pre-commit-config.yaml file,
  • make sure all tests pass by running python -m unittest discover in the root folder of gurobipy-pandas.

After a pull request is submitted, the tests will be run automatically, and the status will appear on the pull request page. If the tests failed, there is a link which can be used to debug the failed tests.

Code reviews

The pull request author should respond to all comments received. If the comment has been accepted and appropriate changes applied, the author should respond by a short message such as "Done" and then resolve the comment. If more discussion is needed on a comment, it should remain open until a solution can be figured out.

Merging changes

Explicit approval and passing tests are required before merging. The pull request author should always merge via "Squash and Merge" and the remote pull request branch should be deleted.