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

script/test: add missing whitespace #5479

Merged
merged 1 commit into from Oct 18, 2016

Conversation

Projects
None yet
4 participants
@ashmaroli
Member

ashmaroli commented Oct 12, 2016

add missing whitespace between -I and test in bundle exec ruby -I test

@DirtyF

DirtyF approved these changes Oct 12, 2016

LGTM

@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Oct 12, 2016

Member

Is this for Windows? This space is not necessary for UNIX systems.

Member

parkr commented Oct 12, 2016

Is this for Windows? This space is not necessary for UNIX systems.

@ashmaroli

This comment has been minimized.

Show comment
Hide comment
@ashmaroli

ashmaroli Oct 13, 2016

Member

@parkr its not required on Windows either.. Just came across this and thought it must have been an overlooked typo..

Member

ashmaroli commented Oct 13, 2016

@parkr its not required on Windows either.. Just came across this and thought it must have been an overlooked typo..

@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Oct 18, 2016

Member

:shrug:

@jekyllbot: merge +dev

Member

parkr commented Oct 18, 2016

:shrug:

@jekyllbot: merge +dev

@jekyllbot jekyllbot merged commit 7aa9c3f into jekyll:master Oct 18, 2016

1 of 2 checks passed

continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

jekyllbot added a commit that referenced this pull request Oct 18, 2016

@ashmaroli ashmaroli deleted the ashmaroli:test-script branch Oct 18, 2016

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