-
-
Notifications
You must be signed in to change notification settings - Fork 225
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
Adjust readme and martin book #1253
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thx for working on this!!
@@ -14,107 +14,6 @@ Additionally, there are [several tools](https://maplibre.org/martin/tools.html) | |||
|
|||
See [Martin book](https://maplibre.org/martin/) for complete documentation. | |||
|
|||
## Installation |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should create some something like a list of important information, and point it to the book - e.g. installation should point directly to the installation section, etc. Just the most important topics, not everything?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, but without the header - just a few links. We may also want to add a "quick start" - makes novice users go there first
README.md
Outdated
* [Installation](https://maplibre.org/martin/installation.html) | ||
* Running with [CLI](https://maplibre.org/martin/run-with-cli.html) or [configuration file](https://maplibre.org/martin/config-file.html) | ||
* [Usage and API](https://maplibre.org/martin/using.html) | ||
<!-- * [Quick Start](https://maplibre.org/martin/quick-Start.html) --> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To bypass the markdwon link check, this line is removed temporally. Will be added after this get merged.
will review shortly. One thing for sure - i don't think we want to use case-sensitive dir name like |
P.S. and same for the filename -- Perhaps a better name is |
Great work, thanks!!! It gave me a good start to continue improving it. Especially big thx for the QGIS doc images, they make things much easier for the novice users. |
Try to fix #1245
Add or keep
Get Started
to readme, it must be easy to take, likecurl -O ..
and thenmartin http://xxx.demo.pmtiles
I guess? ORdocker run ..
with a small http://xxx.demo.pmtiles, do we have some host to serve a demo pmtiles file?License
in readmeMove or remove from
readme.md
Re-use Martin as a library
to bookRunning Martin Service
API
from readme