From 0f9f8bbf0d62679919823da4c3aa0a13b4f35169 Mon Sep 17 00:00:00 2001 From: Vikram Rangnekar Date: Sun, 1 Dec 2019 01:17:27 -0500 Subject: [PATCH] Update CONTRIBUTING.md --- CONTRIBUTING.md | 25 ++++++++++++++----------- 1 file changed, 14 insertions(+), 11 deletions(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index cd4f126..5da61b7 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -8,21 +8,21 @@ Even the GraphQL to SQL compiler that is at the heart of Super Graph is essentia Finally we do have a lot of test for critical parts of the codebase which makes it easy for you to modify with confidence. I'm always available for questions or any sort of guidance so feel fee to reach out over twitter or discord. -* [Getting Started](#get-started) -* [Setting Up the Development Environment](#get-setup) +* [Getting Started](#getting-started) +* [Setting Up the Development Environment](#setup-development-environment) * [Prerequisites](#prerequisites) - * [Get the Super Graph source](#get-source) - * [Start the development envoirnment ](#start-dev) - * [Testing](#testing) + * [Get the Super Graph source](#get-source-code) + * [Start the development envoirnment ](#start-the-development-envoirnment) + * [Testing](#testing-and-linting) * [Contributing](#contributing) * [Guidelines](#guidelines) * [Code style](#code-style) -## Getting Started {#get-started} +## Getting Started - Read the [Getting Started Guide](https://supergraph.dev/guide.html#get-started) -## Setting Up the Development Environment {#get-setup} +## Setup Development Environment ### Prerequisites @@ -30,7 +30,7 @@ Finally we do have a lot of test for critical parts of the codebase which makes - Install [Go 1.13 or above](https://golang.org/doc/install) - Install [Docker](https://docs.docker.com/v17.09/engine/installation/) -### Get the Super Graph source {#get-source} +### Get source code The entire build flow uses `Makefile` there is a whole list of sub-commands you can use to build, test, install, lint, etc. @@ -41,7 +41,7 @@ cd ./super-graph make help ``` -### Start the development envoirnment {#start-dev} +### Start the development envoirnment The entire development flow is packaged into a `docker-compose` work flow. The below `up` command will launch A Postgres database, a example e-commerce app in Rails and Super Graph in development mode. The `db:seed` Rails task will insert sample data into Postgres. @@ -54,7 +54,7 @@ docker-compose up [Super Graph codebase explained](https://supergraph.dev/internals.html) -### Testing and Linting {#testing} +### Testing and Linting ``` make lint test @@ -62,6 +62,8 @@ make lint test ## Contributing +### Guidelines + - **Pull requests are welcome**, as long as you're willing to put in the effort to meet the guidelines. - Aim for clear, well written, maintainable code. - Simple and minimal approach to features, like Go. @@ -72,8 +74,9 @@ make lint test - Leave the code cleaner than when you began. ### Code style + - We're following [Go Code Review](https://github.com/golang/go/wiki/CodeReviewComments). - Use `go fmt` to format your code before committing. - If you see *any code* which clearly violates the style guide, please fix it and send a pull request. No need to ask for permission. - Avoid unnecessary vertical spaces. Use your judgment or follow the code review comments. -- Wrap your code and comments to 100 characters, unless doing so makes the code less legible. \ No newline at end of file +- Wrap your code and comments to 100 characters, unless doing so makes the code less legible.