Thank you for considering contributing to priceproxy
. It's much appreciated, and helps make the tool as useful as possible.
Following these guidelines helps to communicate that you respect the time of the developers managing and developing this open source project. In return, they should reciprocate that respect in addressing your issue, assessing changes, and helping you finalize your pull requests.
priceproxy
is an open source project and we enjoy receiving contributions from our community. There are many ways to contribute, from writing tutorials or blog posts, improving the documentation, submitting bug reports and feature requests or writing code which can be incorporated into the tool itself.
Please don't use the issue tracker for support questions. Use the Vega Community or Discord instead - see the Community section below for links.
Responsibilities:
- Ensure cross-platform compatibility: Linux, MacOSX and Windows.
- Create an issue describing the problem / bug / feature request.
- Create a branch and pull request, and link it to one or more issues.
- Be welcoming to newcomers and encourage diverse new contributors from all backgrounds. See the Python Community Code of Conduct.
Unsure where to begin contributing? You can start by looking through the issue list, or by submitting small pull requests that fix typos or improve documentation
Working on your first Pull Request? You can learn how from this free series, How to Contribute to an Open Source Project on GitHub.
At this point, you're ready to make your changes! Feel free to ask for help; everyone is a beginner at first 😸
If a maintainer asks you to "rebase" your PR, they're saying that a lot of code has changed, and that you need to update your branch so it's easier to merge.
For something that is bigger than a one or two line fix:
- Create your own fork of the code
- Do the changes in your fork
- If you like the change and think the project could use it:
- Be sure you have followed the code style for the project.
- Watch the CI pipeline to make sure all tests pass.
- Submit a pull request.
If you find a security vulnerability, do NOT open an issue. Email [email protected] instead, with "priceproxy security vulnerability" in the Subject line.
In order to determine whether you are dealing with a security issue, ask yourself these two questions:
- Can I access something that's not mine, or something I shouldn't have access to?
- Can I disable something for other people?
When filing an issue, make sure to answer these questions:
- What version of Go are you using (go version)?
- What operating system and processor architecture are you using?
- What did you do?
- What did you expect to see?
- What did you see instead?
General questions related to the Go programming language should go to the golang-nuts
mailing list instead of the issue tracker. The gophers there will answer or ask you to file an issue if you've tripped over a bug.
Please look for an existing issue that matches your feature/enhacement. If you can't find one, open a new issue on our issues list on GitHub which describes the feature you would like to see, why you need it, and how it should work. There are bound to be others out there with similar needs.
The core team welcomes pull requests. PRs that pass the CI pipeline and conform to coding style rules are appreciated.
Style guides:
- Code should be run through
gofmt
. Many text editors and IDEs can do this automatically. - Unit tests are in their own package, e.g. If
bar/foo.go
starts withpackage bar
thenbar/foo_test.go
starts withpackage bar_test
and can only use exported functions from thebar
package. - Code Review Comments