-
Notifications
You must be signed in to change notification settings - Fork 24
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
Add permalinks without .html suffix and fix deprecated config option. #60
Conversation
This is kind of a part of #52. Also, here are the new routes:
|
Definitely like using the non- This looks like the PR pulled in a lot of other CSS changes beyond your description. Was that intended? Haven't read through all of them in detail but I'm betting a few conflict with the Bootstap+FontAwesome PR @nathancarnes put up on #59 (which I was considering getting merged fairly soon). Any thoughts on how best to proceed there? |
Oh, my bad @jasonrclark. I've been working away at the new design. I only meant to PR bcf4102. The design changes I made are not ready. |
I'm going to close this PR, cherry pick the proper changes into a new branch and then PR again. |
Just a small fix, but I've never liked .html in my permalinks. I've also changed
name
from the default toShoes
andpygments: true
tohighlighter: pygments
.