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

Remove warnings for ostruct, csv and syslog #984

Merged
merged 1 commit into from
Feb 5, 2025

Conversation

parthaa
Copy link
Contributor

@parthaa parthaa commented Feb 5, 2025

Before

bash-5.1$ hammer repository list
WARNING: File locale/vi/LC_MESSAGES/hammer_cli_katello.mo outdated, regenerate with 'make all-mo'
/usr/share/ruby/json/common.rb:3: 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.
/opt/app-root/src/hammer-cli-katello/vendor/ruby/3.3.0/gems/hammer_cli-3.13.0/lib/hammer_cli/settings.rb:2: warning: syslog was loaded from the standard library, but will no longer be part of the default gems starting from Ruby 3.4.0.
You can add syslog to your Gemfile or gemspec to silence this warning.
/opt/app-root/src/hammer-cli-katello/vendor/ruby/3.3.0/gems/hammer_cli-3.13.0/lib/hammer_cli/output/adapter.rb:5: warning: csv was loaded from the standard library, but will no longer be part of the default gems starting from Ruby 3.4.0.
You can add csv to your Gemfile or gemspec to silence this warning.

After

bash-5.1$ hammer repository list
WARNING: File locale/vi/LC_MESSAGES/hammer_cli_katello.mo outdated, regenerate with 'make all-mo'

Copy link
Member

@chris1984 chris1984 left a comment

Choose a reason for hiding this comment

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

Ack

@parthaa parthaa merged commit 07ff253 into Katello:main Feb 5, 2025
9 of 11 checks passed
@parthaa parthaa deleted the remove-warnings branch February 5, 2025 22:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants