Skip to content

Fixed some markdown docs - #1625

Merged
lossyrob merged 9 commits into
locationtech:masterfrom
vsimko:master
Sep 29, 2016
Merged

lossyrob merged 9 commits into
locationtech:masterfrom
vsimko:master

Conversation

@vsimko

@vsimko vsimko commented Sep 7, 2016

Copy link
Copy Markdown
Contributor

This pull request contains few fixes in the markdown documentation.
It relates to the Issue #1624

@vsimko vsimko changed the title Fixed Issue #1624 Fixed some markdown docs Sep 7, 2016
Comment thread docs/vector/vector-intro.md Outdated

>"Raster is faster but vector is correcter."
— Somebody
> “Yes raster is faster, but raster is vaster and vector just SEEMS more corrector.”

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is actually in the geotrellis.raster intro! Makes it a funny dialog between geotrellis.vector and geotrellis.raster :)

https://github.com/geotrellis/geotrellis/blob/master/docs/raster/raster-intro.md#geotrellisraster

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I see :)
While reading both docs, I thought somebody forgot to copy the quotation from raster-intro to vector-intro.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Should I revert the change inside vector-intro.md then ?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Sorry about the late reply, we do really appreciate doc changes...things have just been crazy busy.

Yes if you were to back that one change out I would +1 and merge. Thanks!

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yes please.

@lossyrob lossyrob left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

+1 thanks!

@lossyrob
lossyrob merged commit e39bcc1 into locationtech:master Sep 29, 2016
@lossyrob lossyrob added this to the 1.0 milestone Oct 18, 2016
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.

3 participants