-
-
Notifications
You must be signed in to change notification settings - Fork 11
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
Remove check against index basename #47
Conversation
did you run all of the tests? doesn't appear so. this looks like it will break everything we described in the other issues I linked. |
I'd expect so. Just brought attention to the topic in the related ticket. |
At least running the default task on the PR succeeded, no failures. |
|
Unfortunately passing doesn't actually show if the file paths are mutated from expected behavior. I'll have to pull this down and run it in a little bit. thanks |
Thanks, that's what I just wanted to comment. It's a rough removal, I think you need to write a few more tests against the scenarios linked earlier. |
👍 |
I think it will broke other project configuration: permalinks: {
preset: 'pretty'
},
Result: |
I haven't had a chance to look yet, but I think this will require an option. |
Provide PR for #46 research.