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

unit test and CHANGELOG entry for 2870 #2873

Merged
merged 2 commits into from
Sep 30, 2024
Merged

Conversation

fallwith
Copy link
Contributor

@fallwith fallwith commented Sep 30, 2024

unit test branch coverage and a CHANGELOG bugfix writeup to accompany PR #2870

unit test branch coverage and a CHANGELOG bugfix writeup to accompany PR
2870
Copy link

SimpleCov Report

Coverage Threshold
Line 93.8% 93%
Branch 69.62% 50%

@@ -78,6 +78,10 @@ Version 9.14.0 adds Apache Kafka instrumentation for the rdkafka and ruby-kafka

To address a recent Bundler deprecation warning, we started using `Bundler.rubygems.installed_specs` instead of `Bundler.rubygems.all_specs` in environments that seemed appropriate. We discovered the version constraint we used was too low. Now, rather than check the version, we check for the method using `respond_to?`. [PR#2853](https://github.com/newrelic/newrelic-ruby-agent/pull/2853)

- **Bugfix: Support view_component v3.15.0+**
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we list this one on the top summary too?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure, added in ad5f9b8

kaylareopelle
kaylareopelle previously approved these changes Sep 30, 2024
reference the ViewComponent community PR fix in the v9.14.0 summary
@fallwith fallwith merged commit 8fc3ea6 into dev Sep 30, 2024
30 of 31 checks passed
@fallwith fallwith deleted the 2870_test_and_changelog_entry branch September 30, 2024 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants