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

Replace duplicate with postive local test in issue template #5286

Merged
merged 1 commit into from Aug 24, 2016

Conversation

Projects
None yet
5 participants
@DirtyF
Member

DirtyF commented Aug 24, 2016

Fix #5285

@envygeeks

This comment has been minimized.

Show comment
Hide comment
@envygeeks

envygeeks Aug 24, 2016

Contributor

I do like this much better, it nudges and encourages them to consider trying it locally.
LGTM!

❤️

Contributor

envygeeks commented Aug 24, 2016

I do like this much better, it nudges and encourages them to consider trying it locally.
LGTM!

❤️

@envygeeks

This comment has been minimized.

Show comment
Hide comment
@envygeeks

envygeeks Aug 24, 2016

Contributor

LGTM.

Contributor

envygeeks commented Aug 24, 2016

LGTM.

@envygeeks

This comment has been minimized.

Show comment
Hide comment
@envygeeks

envygeeks Aug 24, 2016

Contributor

/cc @jekyll/documentation @jekyll/core

Contributor

envygeeks commented Aug 24, 2016

/cc @jekyll/documentation @jekyll/core

@benbalter

This comment has been minimized.

Show comment
Hide comment
@benbalter

benbalter Aug 24, 2016

Contributor

LGTM. Thanks @DirtyF for keeping it positive. 🌴 🐢 🌵

Contributor

benbalter commented Aug 24, 2016

LGTM. Thanks @DirtyF for keeping it positive. 🌴 🐢 🌵

@envygeeks

This comment has been minimized.

Show comment
Hide comment
@envygeeks

envygeeks Aug 24, 2016

Contributor

@jekyllbot: merge +dev

Contributor

envygeeks commented Aug 24, 2016

@jekyllbot: merge +dev

@jekyllbot jekyllbot merged commit ff55309 into jekyll:master Aug 24, 2016

2 of 3 checks passed

continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
jekyll/lgtm Approved by @envygeeks and @benbalter.

jekyllbot added a commit that referenced this pull request Aug 24, 2016

@DirtyF DirtyF deleted the DirtyF:issue-template branch Aug 24, 2016

@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Aug 24, 2016

Member

❤️

Member

parkr commented Aug 24, 2016

❤️

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