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

chore: set ostruct as a gem dependency #657

Merged
merged 1 commit into from
Jan 7, 2025

Conversation

t27duck
Copy link
Contributor

@t27duck t27duck commented Jan 4, 2025

In Ruby 3.4, requiring ostruct without pulling it in as a gem brings up a deprecation warning. Ostruct is used in the cli and sidekiq load check.

$ bin/bundle exec honeybadger deploy --environment=production --revision=... [email protected]:.../....git'
/home/user/.rbenv/versions/3.4.1/lib/ruby/gems/3.4.0/gems/honeybadger-5.26.1/lib/honeybadger/cli/exec.rb:8: warning: ostruct was loaded from the standard library, but will no longer be part of the default gems starting from Ruby 3.5.0.
You can add ostruct to your Gemfile or gemspec to silence this warning.

Added ostruct as a runtime dependency of the gem which should appease the warning.

Before submitting a pull request, please make sure the following is done:

  1. If you've fixed a bug or added code that should be tested, add tests!
  2. Run rake spec in the repository root.
  3. Use a pull request title that conforms to conventional commits.

In Ruby 3.4, requiring ostruct without pulling it in as a gem brings up a deprecation warning. Ostruct is used in the cli and sidekiq load check.

```
$ bin/bundle exec honeybadger deploy --environment=production --revision=... [email protected]:.../....git'
/home/user/.rbenv/versions/3.4.1/lib/ruby/gems/3.4.0/gems/honeybadger-5.26.1/lib/honeybadger/cli/exec.rb:8: warning: ostruct was loaded from the standard library, but will no longer be part of the default gems starting from Ruby 3.5.0.
You can add ostruct to your Gemfile or gemspec to silence this warning.
```

Added ostruct as a runtime dependency of the gem which should appease the warning.
@t27duck t27duck changed the title chore: Set ostruct as a gem dependency chore: set ostruct as a gem dependency Jan 4, 2025
@joshuap joshuap requested a review from subzero10 January 6, 2025 21:10
Copy link
Member

@stympy stympy left a comment

Choose a reason for hiding this comment

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

Thanks!

@stympy stympy merged commit 360697f into honeybadger-io:master Jan 7, 2025
74 of 75 checks passed
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