Skip to content
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

Allow PreferSafeLogger to migrate uses with level-checks #1842

Merged
merged 2 commits into from
Jul 22, 2021

Conversation

carterkozak
Copy link
Contributor

Oversight in the initial implementation which prevented
level checks from being handled as expected.

After this PR

==COMMIT_MSG==
Allow PreferSafeLogger to migrate logger uses which include level-checks
==COMMIT_MSG==

Oversight in the initial implementation which prevented
level checks from being handled as expected.
@changelog-app
Copy link

changelog-app bot commented Jul 22, 2021

Generate changelog in changelog/@unreleased

Type

  • Feature
  • Improvement
  • Fix
  • Break
  • Deprecation
  • Manual task
  • Migration

Description

Allow PreferSafeLogger to migrate logger uses which include level-checks

Check the box to generate changelog(s)

  • Generate changelog entry

@carterkozak carterkozak requested a review from tpetracca July 22, 2021 17:14
@policy-bot policy-bot bot requested a review from ferozco July 22, 2021 17:14
@bulldozer-bot bulldozer-bot bot merged commit adc6b35 into develop Jul 22, 2021
@bulldozer-bot bulldozer-bot bot deleted the ckozak/fix_isLevelEnabled branch July 22, 2021 17:19
@svc-autorelease
Copy link
Collaborator

Released 4.3.0

bulldozer-bot bot pushed a commit to palantir/witchcraft-api that referenced this pull request Jul 22, 2021
###### _excavator_ is a bot for automating changes across repositories.

Changes produced by the roomba/latest-baseline-oss check.

# Release Notes
## 4.2.0
| Type | Description | Link |
| ---- | ----------- | ---- |
| Improvement | Support the new SafeLogger API | palantir/gradle-baseline#1834 |
| Improvement | Fix the formatting of `baseline-class-uniqueness.lock` files when more than one configuration is listed. A newline was missing. This may require running `./gradlew checkClassUniqueness --write-locks` to update the files. | palantir/gradle-baseline#1838 |


## 4.3.0
| Type | Description | Link |
| ---- | ----------- | ---- |
| Fix | Allow `PreferSafeLogger` to migrate logger uses which include level-checks | palantir/gradle-baseline#1842 |



To enable or disable this check, please contact the maintainers of Excavator.
This was referenced Jul 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants