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

Bring documentation on 'Directory Structure' up-to-date #5573

Merged
merged 3 commits into from Nov 29, 2016

Conversation

Projects
None yet
4 participants
@ashmaroli
Member

ashmaroli commented Nov 12, 2016

The documentation on 'Directory Structure' contains a few dated information and errors that can confuse a new Jekyll user. This pull fixes those errors.

  • Add a note on differing directory structure for sites created with Jekyll v3.2+
  • Add a note about _sass directory contents.
  • Fix any remaining typos or errors in general.

--

Screenshot of the new info panel:

dirst

--
/cc @jekyll/documentation

@DirtyF

This comment has been minimized.

Show comment
Hide comment
@DirtyF

DirtyF Nov 12, 2016

Member

Thanks for bringing this up @ashmaroli, already had to explain this several times to some 😕 users

I tried to make it easier to grasp:

theme-directory-structure

/cc @dret

Member

DirtyF commented Nov 12, 2016

Thanks for bringing this up @ashmaroli, already had to explain this several times to some 😕 users

I tried to make it easier to grasp:

theme-directory-structure

/cc @dret

@DirtyF

DirtyF approved these changes Nov 12, 2016

@ashmaroli

This comment has been minimized.

Show comment
Hide comment
@ashmaroli

ashmaroli Nov 13, 2016

Member

Thank you @DirtyF, 😃 it reads better now.. I found few places that needed some minor edits and have made those..

dirst2

Member

ashmaroli commented Nov 13, 2016

Thank you @DirtyF, 😃 it reads better now.. I found few places that needed some minor edits and have made those..

dirst2

@parkr

parkr approved these changes Nov 29, 2016

@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Nov 29, 2016

Member

@jekyllbot: merge +site

Member

parkr commented Nov 29, 2016

@jekyllbot: merge +site

@jekyllbot jekyllbot merged commit e762b4f into jekyll:master Nov 29, 2016

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 Nov 29, 2016

@ashmaroli ashmaroli deleted the ashmaroli:directory-docs branch Nov 29, 2016

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