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 refactoring #231

Merged
merged 1 commit into from
Dec 10, 2019
Merged

README refactoring #231

merged 1 commit into from
Dec 10, 2019

Conversation

pesekon2
Copy link
Contributor

@pesekon2 pesekon2 commented Dec 3, 2019

Follow Markdown recommended style guides in README:

  • Do not mix different header styles (minuses and hash marks were used for the same level)
  • Line length = 80 characters (I ignored the code excerpts parts)
  • Do not use plain URLs (use <URL> for them)
  • Delete unnecessary indentation
  • Surround fenced code blocks by blank lines
  • Delete multiple consecutive blank lines

Copy link
Member

@ninsbl ninsbl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, looks good to me!


Want to become a core developer? See
[Procedure for gaining Git write access](https://trac.osgeo.org/grass/wiki/HowToContribute#WriteaccesstotheGRASScorerepository)

How to compile GRASS?
---------------------
## How to compile GRASS?

> See INSTALL file.

Yes, you should really read INSTALL.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To make it easier, I would put the link to this INSTALL file here and maybe also to the wiki: https://grasswiki.osgeo.org/wiki/Compile_and_Install

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point. I will do it in the next commit. I also think that there should be noted how to open a ticket/why the issues are still not available on git in the README.

@pesekon2 pesekon2 merged commit d67d646 into OSGeo:master Dec 10, 2019
@pesekon2 pesekon2 deleted the readme-refactoring branch December 10, 2019 12:09
neteler pushed a commit that referenced this pull request Dec 12, 2019
Follow Markdown recommended style guides in README:

* Do not mix different header styles (minuses and hash marks were used for the same level)
* Line length = 80 characters (I ignored the code excerpts parts)
* Do not use plain URLs (use <URL> for them)
* Delete unnecessary indentation
* Surround fenced code blocks by blank lines
* Delete multiple consecutive blank lines
landam pushed a commit to landam/grass that referenced this pull request Jan 28, 2020
Follow Markdown recommended style guides in README:

* Do not mix different header styles (minuses and hash marks were used for the same level)
* Line length = 80 characters (I ignored the code excerpts parts)
* Do not use plain URLs (use <URL> for them)
* Delete unnecessary indentation
* Surround fenced code blocks by blank lines
* Delete multiple consecutive blank lines
petrasovaa pushed a commit to petrasovaa/grass that referenced this pull request Feb 18, 2020
Follow Markdown recommended style guides in README:

* Do not mix different header styles (minuses and hash marks were used for the same level)
* Line length = 80 characters (I ignored the code excerpts parts)
* Do not use plain URLs (use <URL> for them)
* Delete unnecessary indentation
* Surround fenced code blocks by blank lines
* Delete multiple consecutive blank lines
@neteler neteler added this to the 8.0.0 milestone Dec 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants