Skip to content

Bump to JRuby 9.3.14.0, reenable test #9360

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

Merged
merged 1 commit into from
May 21, 2024

Conversation

mhashizume
Copy link
Contributor

This commit updates JRuby to 9.3.14.0 in tests. This version of JRuby addresses a bug with Dir.glob (see jruby/jruby@5b06409) that we had previously worked around in both code and tests (see 663062a). Because the Dir.glob bug has now been addressed, this commit also removes the workarounds we had added.

@mhashizume mhashizume requested a review from a team as a code owner May 21, 2024 17:43
@mhashizume
Copy link
Contributor Author

This should supercede #9359

@mhashizume mhashizume added the maintenance Maintenance chores are excluded from changelogs label May 21, 2024
@mhashizume mhashizume force-pushed the maint/7.x/jruby-dir-glob branch from f802266 to 756979c Compare May 21, 2024 21:17
This commit updates JRuby to 9.3.14.0 in tests. This version of JRuby
addresses a bug with Dir.glob (see jruby/jruby@5b06409) that we had
previously worked around in both code and tests (see 10d219a). Because
the Dir.glob bug has now been addressed, this commit also removes the
workarounds we had added.

Co-authored-by: Shubham Shinde <[email protected]>
@mhashizume mhashizume force-pushed the maint/7.x/jruby-dir-glob branch from 756979c to d000aa6 Compare May 21, 2024 21:18
@tvpartytonight tvpartytonight merged commit 763216b into puppetlabs:7.x May 21, 2024
12 checks passed
@mhashizume mhashizume deleted the maint/7.x/jruby-dir-glob branch May 21, 2024 22:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Maintenance chores are excluded from changelogs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants