Use `assert_nil` instead of `assert_equal nil` #5652

Merged
merged 1 commit into from Dec 10, 2016

Conversation

Projects
None yet
3 participants
@jekyllbot
Contributor

jekyllbot commented Dec 10, 2016

PR automatically created for @pathawks.

Use `assert_nil` instead of `assert_equal nil`

Fixes #5648

@pathawks pathawks changed the title from Use `assert_nil` instead of `assert_equal nil` Fixes #5648 to Use `assert_nil` instead of `assert_equal nil` Dec 10, 2016

@pathawks pathawks added the tests label Dec 10, 2016

@pathawks

This comment has been minimized.

Show comment
Hide comment
Member

pathawks commented Dec 10, 2016

Fixes #5648

/cc @jekyll/core

@pathawks pathawks requested a review from parkr Dec 10, 2016

@parkr

parkr approved these changes Dec 10, 2016

Thank you!!

@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Dec 10, 2016

Member

@pathawks Feel free to merge +dev. :)

Member

parkr commented Dec 10, 2016

@pathawks Feel free to merge +dev. :)

@pathawks

This comment has been minimized.

Show comment
Hide comment
@pathawks

pathawks Dec 10, 2016

Member

@jekyllbot: merge +dev

Member

pathawks commented Dec 10, 2016

@jekyllbot: merge +dev

@jekyllbot jekyllbot merged commit b6b0fdc into master Dec 10, 2016

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 Dec 10, 2016

jekyllbot added a commit that referenced this pull request Dec 10, 2016

parkr added a commit that referenced this pull request Dec 16, 2016

parkr added a commit that referenced this pull request Dec 16, 2016

Merge branch 'master' into liquid-4
* master: (40 commits)
  Update history to reflect merge of #5658 [ci skip]
  Fix a couple of typos in the docs
  Update history to reflect merge of #5657 [ci skip]
  Update variables.md
  Update history to reflect merge of #5653 [ci skip]
  Improve Permalinks documentation.
  Update history to reflect merge of #5652 [ci skip]
  Use `assert_nil` instead of `assert_equal nil`
  Update history to reflect merge of #5513 [ci skip]
  Ignore symlinked file in windows
  Update history to reflect merge of #5612 [ci skip]
  Update history to reflect merge of #5643 [ci skip]
  Update Core team list in README
  Update history to reflect merge of #5641 [ci skip]
  use backticks for Gemfile for consistency since in the next sentence _config.yml file has backtick
  add a set of steps in site_configuration.feature
  update documentation for Windows
  narrow it down to only Windows
  revert and adjust site_configuration.feature
  add 'tzinfo-data' gem to generated Gemfile
  ...

@pathawks pathawks referenced this pull request Jan 5, 2017

Merged

Use `assert_nil` #5725

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