search

zbrox / anpu-zola-theme

The Anpu theme for the Zola static site generator
https://anpu-zola-theme.vercel.app
MIT License
65 stars 34 forks source link

Change README theme instructions #3

Closed alex-tan closed 3 years ago

alex-tan commented 3 years ago

If you do a git clone as the instructions point out, the directory should be named in themes as anpu-zola-theme not anpu. Otherwise zola will complain with 

No `theme.toml` file found. Is the `theme` defined in your `config.toml` present in the `themes` directory and does it have a `theme.toml` inside?`
zbrox commented 3 years ago

Hey, thanks for this, it's a good catch. I should probably be a bit more verbose that the configuration needs to match the path. However I would rather leave the name of the theme as plain anpu and instead add a clarification to clone the repo in a directory named anpu and explain that that path is what needs to be put in the theme configuration (if people want to change it to something arbitrary).

alex-tan commented 3 years ago

👍 No problem, that makes sense.

zbrox commented 3 years ago

@alex-tan should I close this or do you still want to expand the readme?

alex-tan commented 3 years ago

Oh, my bad I thought you were going to change it. I updated the pull request to instruct how to git clone so the directory name matches the theme name.

zbrox commented 3 years ago

Great, thank you!