Skip to content
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

not need to accept #193

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
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
12 changes: 6 additions & 6 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,19 +3,19 @@ How to Contribute

## Solutions

Please **do not send pull requests for replacing my code**. If you'd like to contribute [solutions](https://github.com/thekarangoel/Projects-Solutions), please see the following instructions:
Please **do not send pull requests to replace my code**. If you'd like to contribute [solutions](https://github.com/thekarangoel/Projects-Solutions), please see the following instructions:

- Solve the problem, and keep it in your repo. DO NOT send merge request yet.
- Solve the problem, and keep it in your repo. DO NOT send a merge request yet.

- Find the problem in this [README](https://github.com/thekarangoel/Projects-Solutions/blob/master/README.md) file.

- Edit the file in this format (notice the double square brackets):

\**This is a problem** - And here goes it's description. \[[octocat (language)]]\(link-to-octocats-solution) \[[your_username (C++)]]\(link-to-your-solution)
\**This is a problem** - And here goes it's description. \[[octocat (language)]]\(link-to-autocrats-solution) \[[your_username (C++)]]\(link-to-your-solution)

This results in something like this:

**This is a problem** - And here goes it's description. [[octocat (language)]](link-to-octocats-solution) [[your_username (C++)]](link-to-your-solution)
**This is a problem** - And here goes its description. [[octocat (language)]](link-to-autocrats-solution) [[your_username (C++)]](link-to-your-solution)

- Please respect other people and do not delete or move anyone else's link. Place your solution at the end of the list of solutions.

Expand All @@ -28,6 +28,6 @@ Please **do not send pull requests for replacing my code**. If you'd like to con

If you have suggestions for adding or removing projects, feel free to [open an issue](https://github.com/thekarangoel/Projects/issues/new) to discuss it, or directly create a pull request after you edit the *README.md* file with necessary changes. If you add a problem, make sure you add it in the correct category.

It is required that any changes made to the projects (this does not include adding solutions) in *README.md*, must be reflected in the [Solutions *README.md* file](https://github.com/thekarangoel/Projects-Solutions/blob/master/README.md)
It is required that any changes made to the projects (this does not include adding solutions) be in *README.MD*, must be reflected in the [Solutions *README.md* file](https://github.com/thekarangoel/Projects-Solutions/blob/master/README.md)

If you're copying a project from some other webpage, you **MUST** link to it in the *Sources* section at the bottom of repo's *README.md* and *README-scratch.md*.
If you're copying a project from some other webpage, you **MUST** link to it in the *Sources* section at the bottom of the repo's *README.md* and *README-scratch. md*.