Skip to content

Latest commit

 

History

History
42 lines (27 loc) · 2.66 KB

CONTRIBUTING.md

File metadata and controls

42 lines (27 loc) · 2.66 KB

Reporting Issues with xDrip

Before opening an issue, please communicate with us by posting in the support channels:
https://www.facebook.com/groups/xDripG5
https://gitter.im/jamorham/xDrip-plus
We will encourage you to open an issue if it is not already known and there is no work-around.

Please try to provide as much information as possible when raising an issue:

  • If it is a bug, please try to explain how to reproduce it, roughly which settings you are using, what you are seeing or not seeing. Screenshots welcomed.
  • If it relates to a new feature, please explain the rationale for the feature, use cases and detail your suggestion.

Contributing patches to xDrip

Do you have an idea for a feature or enhancement or bugfix for Nightscout xDrip?

Patches are welcomed! To ensure the best all round results they need be a good fit with the project roadmap, the ecosystem of users and respect overall project stability and good practice.

The best way to get patches accepted is to discuss your ideas with project maintainers prior to implementation and discuss how to structure things so that they can fit well within the project and also the rationale for the change.

Project reviewers are likely to use criteria similar to that outlined below:

  • Does the patch affect any existing functionality? What is the impact of the change?
  • Does the patch enhance the app in a way which is proportional to the change being made?
  • Can any significant behavioral change be controlled with a settings switch without excessively increasing the user interface complexity?
  • Is there an existing maintainer working on the affected area with possibly parallel activity?
  • Are there any other potential negative consequences to the change?
  • Does the patch contain any unrelated changes?
  • Has the patch been properly tested?

Make sure any pull request has a clear title and a good description.

Try not to bundle different patches together and keep the change set to the minimum required. Even unrelated changes to whitespace have to be reviewed in case a random keystroke broke something. The less lines there are to review, the clearer the patch will be to any reviewer.

Maintainers are not always immediately aware of pull request submissions, which is another reason why it helps to have discussed your work prior to submission. Feel free to ask about the progress of processing your PR as maintainers may not realise a PR is awaiting action.

Translations

Please use xDrip+ on Crowdin if you have any questions about translations, or like to add a language or contribute to an existing language.


Nightscout xDrip is maintained by volunteers.