Use case-insensitive checking for the file extensions when loading config files #2718

Merged
merged 2 commits into from Aug 10, 2014

Conversation

Projects
None yet
3 participants
@parkr
Member

parkr commented Aug 10, 2014

No description provided.

@alfredxing

This comment has been minimized.

Show comment
Hide comment
@alfredxing

alfredxing Aug 10, 2014

Member

👍
By the way, do we really need the capture group around the a in /\.y(a)?ml/? I think /\.ya?ml/ would do the same thing.

Member

alfredxing commented Aug 10, 2014

👍
By the way, do we really need the capture group around the a in /\.y(a)?ml/? I think /\.ya?ml/ would do the same thing.

@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Aug 10, 2014

Member

By the way, do we really need the capture group around the a in /.y(a)?ml/? I think /.ya?ml/ would do the same thing.

Good point! I'll update it.

Member

parkr commented Aug 10, 2014

By the way, do we really need the capture group around the a in /.y(a)?ml/? I think /.ya?ml/ would do the same thing.

Good point! I'll update it.

@parkr parkr merged commit e5463b4 into master Aug 10, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details

@parkr parkr deleted the case-insensitive-extnames branch Aug 10, 2014

parkr added a commit that referenced this pull request Aug 10, 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.