SCGLIMS is open-source and, as such, grows (or shrinks) & improves in part due to the community. Below are some guidelines on how to help with the project. They are based on the guidelines of django-import-export.
- is GPL-licensed. All contributed code must be either
- the original work of the author, contributed under the GPL, or...
- work taken from another project released under a GPL-compatible license.
- SCGLIMS' git master branch should always be stable, production-ready & passing all tests
So you've found a bug or have a great idea for a feature. Here's the steps you should take to help get it added/fixed in SCGLIMS:
- First, check to see if there's an existing issue/pull request for the bug/feature. All issues are at https://github.com/bils/scglims/issues and pull reqs are at https://github.com/bils/scglims/pulls.
- If there isn't one there, please file an issue. The ideal report includes:
- A description of the problem/suggestion.
- How to recreate the bug.
- If relevant, including the versions of your:
- Python interpreter
- Django
- Optionally of the other dependencies involved
- Ideally, creating a pull request with a (failing) test case demonstrating what's wrong. This makes it easy for us to reproduce & fix the problem.
If you're ready to take the plunge & contribute back some code/docs, the process should look like:
- Fork the project on GitHub into your own account.
- Clone your copy of SCGLIMS.
- Make a new branch in git & commit your changes there.
- Push your new branch up to GitHub.
- Again, ensure there isn't already an issue or pull request out there on it. If there is & you feel you have a better fix, please take note of the issue number & mention it in your pull request.
- Create a new pull request (based on your branch), including what the problem/feature is, versions of your software & referencing any related issues/pull requests.
In order to be merged into django-import-export, contributions must have the following:
- A solid patch that:
- is clear.
- works across all supported versions of Python/Django.
- follows the existing style of the code base (mostly PEP-8).
- comments included as needed to explain why the code functions as it does
- A test case that demonstrates the previous flaw that now passes with the included patch.
- If it adds/changes a public API, it must also include documentation for those changes.
- Must be appropriately licensed (see Philosophy).
- Add yourself to the AUTHORS file.
If your contribution lacks any of these things, they will have to be added by a core contributor before being merged into SCGLIMS properly.