Skip to content
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

Prefer .yaml over .toml #5966

Merged
merged 1 commit into from Mar 20, 2017
Merged

Prefer .yaml over .toml #5966

merged 1 commit into from Mar 20, 2017

Conversation

fulldecent
Copy link
Contributor

YAML is mature and already works with Jekyll. This PR recommends YAML as first choice and TOML as second in error messages.

TOML is v0.4.0 and is "changing a lot" according to its author.

And for the love of god, will somebody please read XKCD 927.

@ghost
Copy link

ghost commented Mar 20, 2017

is this a +doc merge?

@DirtyF
Copy link
Member

DirtyF commented Mar 20, 2017

@pnn I would label this as +minor+dev

@ghost
Copy link

ghost commented Mar 20, 2017

@DirtyF necessitating a minor version change for a recommendation change?

@DirtyF
Copy link
Member

DirtyF commented Mar 20, 2017

@pnn yeah, you're right, minor is not appropriated here and it's not a bug either, so that leaves us with dev I guess.

@ghost
Copy link

ghost commented Mar 20, 2017

alright!

@ghost
Copy link

ghost commented Mar 20, 2017

@jekyllrb: merge +dev

@jekyllbot jekyllbot merged commit ac1369c into jekyll:master Mar 20, 2017
jekyllbot added a commit that referenced this pull request Mar 20, 2017
@ghost
Copy link

ghost commented Mar 20, 2017

huh, didn't know @jekyllbot catches my mistakes

@jekyll jekyll locked and limited conversation to collaborators Jul 11, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants