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

refact: code cleanup #79

Merged
merged 6 commits into from
Jan 15, 2025
Merged

refact: code cleanup #79

merged 6 commits into from
Jan 15, 2025

Conversation

sebthom
Copy link
Contributor

@sebthom sebthom commented Jan 10, 2025

I performed minor code improvements as part of general code gardening. To enhance clarity and maintainability, each type of change has been isolated into its own commit.

@headius
Copy link
Member

headius commented Jan 11, 2025

Welcome and thanks for your PRs! I did not have a change to review today but I'll get to it over the weekend or early next week!

@sebthom
Copy link
Contributor Author

sebthom commented Jan 11, 2025

@headius for context I am core maintainer of the Eclipse TM4E project where we heavily rely on joni.

@headius
Copy link
Member

headius commented Jan 15, 2025

@sebthom That's exciting to hear! I will review your PRs now and see about getting a release soon.

@headius
Copy link
Member

headius commented Jan 15, 2025

Looks good!

@headius headius merged commit 2044540 into jruby:master Jan 15, 2025
1 check passed
@headius headius added this to the 2.2.2 milestone Jan 15, 2025
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.

2 participants