Skip to content

Changed some minor typos in Contributing.md #150

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 27, 2019
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 9 additions & 9 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
@@ -1,35 +1,35 @@
# Contribution Guide

Thanks for taking interest and I appreciate your efforts towards making this projects even better.
Thanks for taking interest and I appreciate your efforts towards making this project even better.

## How to setup?

This is the most simple project when it comes to contributions, setup, opening issues/pull requests. So et's get started.
This is the most simple project when it comes to contributions, setup, opening issues/pull requests. So let's get started.

- Clone the repo using the command `git clone git@github.com:knaxus/problem-solving-javascript.git`<sup>1</sup>
- Install the packages to get suport for linter using `npm install`
- Install the packages to get support for linter using `npm install`

1: If you do not have **ssh** setup for github, while cloning go with **https**

### Before you start, keep the following things in mind:
- We use ESLint for code linting
- The linter follows [Airbnb JavaScript Style Guide](https://github.com/airbnb/javascript)
- Go though the folder structure carefully and follow the same
- Go though the format and file convenetions used while adding tests (both test case and test files)
- Go through the folder structure carefully and follow the same
- Go through the format and file convenetions used while adding tests (both test case and test files)

## Adding your code

- When adding a new problem with solution
- When adding a new problem with a solution
- Take care of the filename convention (Very Important)
- Problem statement should be there and support it with some examples
- Make sure you've add the **Run Time Complexity** of your solution
- A problem statement should be there and support it with some examples
- Make sure you've added the **Run Time Complexity** of your solution
- Please take care of the segregation of the Problems as per the given Folder Structure
- It's great if you can add the Unit Tests to verify your solutions as well

- When adding a Unit Test
- Take care of the file name convention
- Make sure CI (Travis) is passing
- Expanding the test suite until (close to) 100 percent code coverage is achieved
- Expanding the test suite until (close to) 100 percentage code coverage is achieved

### Notes

Expand Down