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

corrected reference to layout in index.md #6032

Merged
merged 1 commit into from Apr 17, 2017

Conversation

Projects
None yet
3 participants
@meyer-sven
Contributor

meyer-sven commented Apr 17, 2017

The file index.md referenced the layout "default.html" in its frontmatter. As far as I know and according to my tests, a layout has to be referenced without its file extension.

This is my first proposal for a file change in github - sorry, if I did something wrong. Your tutorial was very helpful to me for starting my first Jekyll-blog.

corrected reference to layout in index.md
The file index.md referenced the layout "default.html" in its frontmatter. As far as I know and according to my tests, a layout has to be referenced without its file extension.

This is my first proposal for a file change in github - sorry, if I did something wrong. Your tutorial was very helpful to me for starting my first Jekyll-blog.
@DirtyF

DirtyF approved these changes Apr 17, 2017

Thanks for your feedback!

You're totally right, layout does not need the html extension, good catch! 👌

@DirtyF

This comment has been minimized.

Show comment
Hide comment
@DirtyF

DirtyF Apr 17, 2017

Member

@jekyllbot: merge +docs

Member

DirtyF commented Apr 17, 2017

@jekyllbot: merge +docs

@jekyllbot jekyllbot merged commit a2e5899 into jekyll:master Apr 17, 2017

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

jekyllbot added a commit that referenced this pull request Apr 17, 2017

@meyer-sven meyer-sven deleted the meyer-sven:patch-1 branch Apr 18, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment