Skip to content

Conversation

@johnnysohn
Copy link

@johnnysohn johnnysohn commented Jan 3, 2025

Description

This is the port of this PR #1 on branch 1.4.x, in preparation for enabling logback 1.4 to be used at HubSpot

Copy link

@cath0htac cath0htac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

Edit: Are there any relevant tests we can add a test case to?

Copy link

@jaredstehler jaredstehler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm 🚢

@jaredstehler jaredstehler merged commit 5a9ae8f into branch_1.4.x_hubspot Jan 6, 2025
1 check passed
@jaredstehler jaredstehler deleted the js_logback_yammer_metrics branch January 6, 2025 15:11
@jaredstehler jaredstehler restored the js_logback_yammer_metrics branch January 6, 2025 15:11
@jaredstehler jaredstehler deleted the js_logback_yammer_metrics branch January 6, 2025 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants