e5b44f4bd3
* Update CONTRIBUTING.md for dependency update methods (go dep -> go mod) Signed-off-by: Yong Tang <yong.tang.github@outlook.com> * Small fix for reveiw comment Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
2.8 KiB
Contributing to CoreDNS
Welcome! Our community focuses on helping others and making CoreDNS the best it can be. We gladly accept contributions and encourage you to get involved!
Bug Reports
First, please search this repository with a variety of keywords to ensure your bug is not already reported.
If not, open an issue and answer the questions so we can understand and reproduce the problematic behavior.
The burden is on you to convince us that it is actually a bug in CoreDNS. This is easiest to do when you write clear, concise instructions so we can reproduce the behavior (even if it seems obvious). The more detailed and specific you are, the faster we will be able to help you. Check out How to Report Bugs Effectively.
Please be kind. 😄 Remember that CoreDNS comes at no cost to you, and you're getting free help.
Minor Improvements and New Tests
Submit pull requests at any time. Make sure to write tests to assert your change is working properly and is thoroughly covered.
New Features
First, please search with a variety of keywords to ensure your suggestion/proposal is new.
Please also check for existing pull requests to see if someone is already working on this. We want to avoid duplication of effort.
If the proposal is new and no one has opened pull request yet, you may open either an issue or a pull request for discussion and feedback.
If you are going to spend significant time implementing code for a pull request, best to open an issue first and "claim" it and get feedback before you invest a lot of time.
If someone already opened a pull request, but you think the pull request has stalled and you would like to open another pull request for the same or similar feature, get some of the maintainers (see OWNERS) involved to resolve the situation and move things forward.
If possible make a pull request as small as possible, or submit multiple pull request to complete a feature. Smaller means: easier to understand and review. This in turn means things can be merged faster.
Updating Dependencies
We use Go Modules as the tool to manage vendor dependencies.
Use the following to update the version of all dependencies
$ go get -u
After the dependencies have been updated or added, you might run the following to cleanup the go module files:
$ go mod tidy
Please refer to Go Modules for more details.
Thank You
Thanks for your help! CoreDNS would not be what it is today without your contributions.