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

Use `assert_nil` #5725

Merged
merged 1 commit into from Jan 9, 2017

Conversation

Projects
None yet
5 participants
@jekyllbot
Contributor

jekyllbot commented Jan 5, 2017

PR automatically created for @pathawks.

Use `assert_nil`
@pathawks

This comment has been minimized.

Show comment
Hide comment
@pathawks

pathawks Jan 5, 2017

Member

Not sure how I missed these in #5652

Member

pathawks commented Jan 5, 2017

Not sure how I missed these in #5652

@pathawks

This comment has been minimized.

Show comment
Hide comment
@pathawks
Member

pathawks commented Jan 5, 2017

@oe

oe approved these changes Jan 6, 2017

@DirtyF

DirtyF approved these changes Jan 6, 2017

@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Jan 9, 2017

Member

@jekyllbot: merge +dev

Member

parkr commented Jan 9, 2017

@jekyllbot: merge +dev

@jekyllbot jekyllbot merged commit d2eb645 into master Jan 9, 2017

2 checks passed

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

@jekyllbot jekyllbot deleted the pull/assert_nil branch Jan 9, 2017

jekyllbot added a commit that referenced this pull request Jan 9, 2017

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