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

Enable Leak Detection in CI Pipeline #242

Merged
merged 4 commits into from
Feb 22, 2024
Merged

Enable Leak Detection in CI Pipeline #242

merged 4 commits into from
Feb 22, 2024

Conversation

jchrys
Copy link
Collaborator

@jchrys jchrys commented Feb 20, 2024

Motivation:
Address lack of ByteBuffer leak detection during development to prevent potential production issues.

Modifications:
Added leak detection feature to CI pipeline for automatic detection during testing.

Result:
Better CI
Resolves #241

@jchrys jchrys added the enhancement New feature or request label Feb 20, 2024
@jchrys jchrys marked this pull request as ready for review February 20, 2024 18:33
Motivation:
Address lack of ByteBuffer leak detection during development to prevent potential production issues.

Modifications:
Added leak detection feature to CI pipeline for automatic detection during testing.

Result:
Better CI
Resolves #241
Copy link
Contributor

@mirromutth mirromutth left a comment

Choose a reason for hiding this comment

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

LGTM

BTW, I prefer to have a newline at the end of text files

@jchrys
Copy link
Collaborator Author

jchrys commented Feb 22, 2024

LGTM

BTW, I prefer to have a newline at the end of text files

Let me add it :D

@jchrys jchrys added this to the 1.1.2 milestone Feb 22, 2024
@jchrys jchrys merged commit 068c0e3 into trunk Feb 22, 2024
15 checks passed
@jchrys jchrys deleted the 1x-leak branch February 22, 2024 06:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feature]Enable Leak detection in CI pipeline
2 participants