Skip to content

logger: Allow 1.5 - #367

Merged
pcai merged 1 commit into
WinRb:mainfrom
bastelfreak:logger
Oct 13, 2026
Merged

pcai merged 1 commit into
WinRb:mainfrom
bastelfreak:logger

Conversation

@bastelfreak

@bastelfreak bastelfreak commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

1.5 is the version of the vendored gem in Ruby 3.2. winrm doesn't use any 1.6 specific logic. By lowering the version to 1.5, we can use the bundled version in Ruby 3.2 and don't have to install additional gems.

See https://stdgems.org/3.2.11/ for details.

What kind of change is this?

allow loger 1.5 dependency

Did you add tests for your changes?

yes, works for me

Summary of changes

Other information

1.5 is the version of the vendored gem in Ruby 3.2. winrm doesn't use
any 1.6 specific logic. By lowering the version to 1.5, we can use the
bundled version in Ruby 3.2 and don't have to install additional gems.

See https://stdgems.org/3.2.11/ for details.
@pcai
pcai merged commit ec912d6 into WinRb:main Oct 13, 2026
11 checks passed
@pcai

pcai commented Oct 13, 2026

Copy link
Copy Markdown
Contributor

@bastelfreak thank you

@bastelfreak
bastelfreak deleted the logger branch October 13, 2026 06:25
@bastelfreak

Copy link
Copy Markdown
Contributor Author

@pcai thanks for the quick merge! Any chance you can create a new release?

@pcai pcai mentioned this pull request Oct 13, 2026
pcai added a commit that referenced this pull request Oct 13, 2026
Cut 2.4.2 with the changes that landed since 2.4.1.

- Replace erubi with the Ruby standard library ERB; the erubi runtime dependency is gone (#366)
- Allow the `logger` dependency at version 1.5, so Ruby 3.2 users can use the bundled gem (#367)

Bumps the version to 2.4.2 and records both changes in CHANGELOG.md.
@pcai

pcai commented Oct 13, 2026

Copy link
Copy Markdown
Contributor

All set

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.

3 participants