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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a Pro Tip to docs about front matter variables being optional #1147

Merged
merged 1 commit into from Jun 3, 2013

Conversation

Projects
None yet
4 participants
@beeeswax
Contributor

beeeswax commented May 25, 2013

Added a note to Front Matter in the docs about empty YAML.

I use (and have seen other people use) layout=nil to get jekyll to process things that aren't pages, most commonly atom.xml, but after testing it just now in 1.0, the user can just leave the YAML empty and still get Liquid processing...

Thanks! 馃槃
MM

@parkr

This comment has been minimized.

Member

parkr commented Jun 2, 2013

Cool tip, I'm down to add it to the site. @mattr-?

In the meantime, please update your pull-request such that it uses the page-based docs site 鈥 this page is now in /site/docs/frontmatter.md :)

@mattr-

This comment has been minimized.

Member

mattr- commented Jun 3, 2013

Awesome tip!

@beeeswax If you wouldn't mind, could you rebase this so that we could merge it automatically? Thanks!

@parkr

This comment has been minimized.

Member

parkr commented Jun 3, 2013

screen shot 2013-06-03 at 5 44 55 pm

This is what it'll look like. Cool with 2 tips in a row?

@parkr

This comment has been minimized.

Member

parkr commented Jun 3, 2013

(And we can manually merge - I don't mind)

@beeeswax

This comment has been minimized.

Contributor

beeeswax commented Jun 3, 2013

I think this is all set now. Happy Monday! 馃憤

@mattr-

This comment has been minimized.

Member

mattr- commented Jun 3, 2013

I like it.

On Mon, Jun 3, 2013 at 10:45 AM, Parker Moore notifications@github.comwrote:

(And we can manually merge - I don't mind)


Reply to this email directly or view it on GitHubhttps://github.com//pull/1147#issuecomment-18850394
.

parkr added a commit that referenced this pull request Jun 3, 2013

Merge pull request #1147 from beeeswax/frontmatter-nil
Add a Pro Tip to docs about front matter variables being optional

@parkr parkr merged commit 94e5cb1 into jekyll:master Jun 3, 2013

1 check passed

default The Travis CI build passed
Details

parkr added a commit that referenced this pull request Jun 3, 2013

@jekyll jekyll locked and limited conversation to collaborators Feb 27, 2017

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