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 rake task to generate default config doc #2456

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@ivantsepp
Contributor

ivantsepp commented May 27, 2014

Continuation of #2452. Let me know what you think!

Some changes from the original documentation:

  • destination, plugins, and layouts began with ./, but now those are stripped.
  • encoding, relative_permalinks, and paginate_path now have the correct values (they were not up to date before)
  • url does not exist anymore (I don't think Jekyll used this)
  • watch, server, and paginate are the deprecated/common issues. Those are removed.
  • Spacing
  • Order
Ivan Tse
Add rake task to generate default config doc
This is to autogenerate the default configuration that is displayed in
the documentation.
@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Aug 6, 2014

Member

I don't like the way this looks, with everything squished together. I'd rather just manually keep them all in line. Thank you for this though!

Member

parkr commented Aug 6, 2014

I don't like the way this looks, with everything squished together. I'd rather just manually keep them all in line. Thank you for this though!

@parkr parkr closed this Aug 6, 2014

@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Aug 6, 2014

Member

A script that would read in the YAML from site/_docs/configuration.md and compare it against Jekyll::Configuration::DEFAULTS would be helpful, though.

Member

parkr commented Aug 6, 2014

A script that would read in the YAML from site/_docs/configuration.md and compare it against Jekyll::Configuration::DEFAULTS would be helpful, though.

@ivantsepp

This comment has been minimized.

Show comment
Hide comment
@ivantsepp

ivantsepp Aug 6, 2014

Contributor

Ah that's a good idea! Would this script be part of travis ci?

Contributor

ivantsepp commented Aug 6, 2014

Ah that's a good idea! Would this script be part of travis ci?

parkr added a commit that referenced this pull request Aug 6, 2014

@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.