From f85f4c174fb9e78c4bd76876b1bef26d57e24a3f Mon Sep 17 00:00:00 2001 From: Kayla McKay <39921134+kaymckay@users.noreply.github.com> Date: Wed, 16 Mar 2022 13:20:05 -0500 Subject: [PATCH] update contributing (#242) --- .github/CONTRIBUTING.md | 61 ++++------------------------------------- 1 file changed, 5 insertions(+), 56 deletions(-) diff --git a/.github/CONTRIBUTING.md b/.github/CONTRIBUTING.md index a1f4d7a4..462634b6 100644 --- a/.github/CONTRIBUTING.md +++ b/.github/CONTRIBUTING.md @@ -1,27 +1,13 @@ # Contributing -We'd love to accept your contributions to this project! There are just a few guidelines you need to follow. - -## Bugs - -Bug reports should be opened up as [issues](https://help.github.com/en/github/managing-your-work-on-github/about-issues) on the [go-vela/community](https://github.com/go-vela/community) repository! - -## Feature Requests - -Feature Requests should be opened up as [issues](https://help.github.com/en/github/managing-your-work-on-github/about-issues) on the [go-vela/community](https://github.com/go-vela/community) repository! - -## Pull Requests - -**NOTE: We recommend you start by opening a new issue describing the bug or feature you're intending to fix. Even if you think it's relatively minor, it's helpful to know what people are working on.** - -We are always open to new PRs! You can follow the below guide for learning how you can contribute to the project! - ## Getting Started +We'd love to accept your contributions to this project! If you are a first time contributor, please review our [Contributing Guidelines](https://go-vela.github.io/docs/community/contributing_guidelines/) before proceeding. + ### Prerequisites * [Review the commit guide we follow](https://chris.beams.io/posts/git-commit/#seven-rules) - ensure your commits follow our standards -* [Golang](https://golang.org/dl/) - for source code and [dependency management](https://github.com/golang/go/wiki/Modules) +* Review our [style guide](https://go-vela.github.io/docs/community/contributing_guidelines/#style-guide) to ensure your code is clean and consistent. ### Setup @@ -57,25 +43,7 @@ git remote add fork https://github.com/your_fork/types cd $HOME/go-vela/types ``` -* Write your code - * Please be sure to [follow our commit rules](https://chris.beams.io/posts/git-commit/#seven-rules) - * Please address linter warnings appropriately. If you are intentionally violating a rule that triggers a linter, please annotate the respective code with `nolint` declarations [[docs](https://golangci-lint.run/usage/false-positives/)]. we are using the following format for `nolint` declarations: - - ```go - // nolint: // - ``` - - Example: - - ```go - // nolint:gocyclo // legacy function is complex, needs simplification - func superComplexFunction() error { - // .. - } - ``` - - Check the [documentation for more examples](https://golangci-lint.run/usage/false-positives/). - +* Write your code and tests to implement the changes you desire. * Test the repository code (ensures your changes don't break existing functionality): @@ -98,25 +66,6 @@ make clean git push fork master ``` -* Open a pull request! - * For the title of the pull request, please use the following format for the title: - - ```text - feat(wobble): add hat wobble - ^--^^------^ ^------------^ - | | | - | | +---> Summary in present tense. - | +---> Scope: a noun describing a section of the codebase (optional) - +---> Type: chore, docs, feat, fix, refactor, or test. - ``` - - * feat: adds a new feature (equivalent to a MINOR in Semantic Versioning) - * fix: fixes a bug (equivalent to a PATCH in Semantic Versioning) - * docs: changes to the documentation - * refactor: refactors production code, eg. renaming a variable; doesn't change public API - * test: adds missing tests, refactors tests; no production code change - * chore: updates something without impacting the user (ex: bump a dependency in package.json or go.mod); no production code change - - If a code change introduces a breaking change, place ! suffix after type, ie. feat(change)!: adds breaking change. correlates with MAJOR in semantic versioning. +* Make sure to follow our [PR process](https://go-vela.github.io/docs/community/contributing_guidelines/#development-workflow) when opening a pull request Thank you for your contribution!