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

Readme #1657

Closed
wants to merge 4 commits into from
Closed

Readme #1657

wants to merge 4 commits into from

Conversation

SatenderRathore
Copy link

I have updated README now its looking a bit good :)

@ThiefMaster
Copy link
Member

First of all, these four commits should be squashed into a single one. Their commit messages are pretty useless but times four even more so. "Format README" would be a better message.

Anyway, @mitsuhiko has a strong preference for a simple plaintext README, so I doubt this PR will be merged.

@ThiefMaster
Copy link
Member

Just noticed that you created lots of PRs that just do minor changes to README files (also in other projects) that don't actually improve anything.

Please, don't do that! It's just noise for the maintainers of those projects. If you want to contribute to a project, pick some issue that's actually useful and try to solve/implement it and create a PR for that.

@davidism davidism closed this Dec 18, 2015
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 14, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants