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

Update license text in README #63

Merged
merged 1 commit into from
Sep 2, 2024
Merged

Conversation

pka
Copy link
Member

@pka pka commented Sep 2, 2024

and fix directory permission in planetiler tutorial.

The "See also" chapter in in README.md should maybe better included in the "Creating Shortbread Vector Tiles" chapter.

@@ -17,6 +17,7 @@ Shortbread Tiles can be created with [Planetiler](https://github.com/onthegomap/
### Quickstart (Docker)

```bash
mkdir data
Copy link

Choose a reason for hiding this comment

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

That seems unrelated to who the licensor is.

Copy link

Choose a reason for hiding this comment

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

Ah, sorry, didn't spot the

and fix directory permission in planetiler tutorial.

in the PR description. Though, while I don't know this project's contribution conventions, it's usually a good idea to submit logically independent changes separately, rather than lumping them together in a single PR. 🙂

@joto joto merged commit 6e1524b into shortbread-tiles:main Sep 2, 2024
1 check passed
pka pushed a commit to pka/shortbread-docs that referenced this pull request Dec 2, 2024
* Add native image processing to cards

* feat: Image processing options, markdown subtitles

* fixing a type

* Public Domain Images

* image height fix

* Smaller Image

* Removed fullwidth param

* Smaller image from unsplash

* Replaced static image as well

* Update single.html

* Update list.html

* Update list.html

* Update single.html

* Update list.html

* Update single.html

* Update cards.md

* Update cards.md

---------

Co-authored-by: Xin <xin@imfing.com>
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.

None yet

3 participants