/jekyll

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Removed deprecation inconsistency #3535

Merged
merged 1 commit into from Mar 2, 2015

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
frozen-due-to-age
Projects
None yet
Milestone
No milestone
4 participants
@JoopAue @parkr @alfredxing @jekyllbot
@JoopAue
Contributor

JoopAue commented Mar 2, 2015

Replaced occasions where the logger was used to report deprecations.
Deprecator is now used instead.

see #3534
@JoopAue JoopAue Removed deprecation inconsistency 
Replaced occasions where the logger was used to report deprecations.
Deprecator is now used instead.

Signed-off-by: Joop Aué <joopaue@gmail.com>

All checks have passed

2 successful checks
ci/circleci — Your tests passed on CircleCI!
Details
continuous-integration/travis-ci/pr — The Travis CI build passed
Details
7a31029
@parkr
Owner

parkr commented Mar 2, 2015

Perfect, thank you! :shipit:

@alfredxing alfredxing added a commit that referenced this pull request Mar 2, 2015

@alfredxing alfredxing Merge pull request #3535 from delftswa2014/deprecation-inconsistency 
Removed deprecation inconsistency
 27a02e1

@alfredxing alfredxing merged commit 27a02e1 into jekyll:master Mar 2, 2015
2 checks passed

2 checks passed

ci/circleci Your tests passed on CircleCI!
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@alfredxing alfredxing added a commit that referenced this pull request Mar 2, 2015

@alfredxing alfredxing Update history to reflect merge of #3535 [ci skip] 8917ef0
@alfredxing
Member

alfredxing commented Mar 2, 2015

Thanks!

MartinRogalla deleted the delftswa2014:deprecation-inconsistency branch Mar 3, 2015

jekyllbot locked and limited conversation to collaborators Feb 27, 2017

jekyllbot added the frozen-due-to-age label Feb 27, 2017

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