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

Add ViewComponent regression test #2411

Merged
merged 5 commits into from
Jan 25, 2024
Merged

Conversation

hannahramadan
Copy link
Contributor

@hannahramadan hannahramadan commented Jan 23, 2024

ViewComponent errors are being suppressed, as #2410 reported. Once the fix from that PR is merged, our new test should pass.

Update: fix merged and new test passing!

@hannahramadan hannahramadan marked this pull request as ready for review January 24, 2024 17:21
fallwith
fallwith previously approved these changes Jan 24, 2024
kaylareopelle
kaylareopelle previously approved these changes Jan 24, 2024
CHANGELOG.md Outdated Show resolved Hide resolved
@@ -1,23 +1,25 @@
# New Relic Ruby Agent Release Notes


## dev
## v9.7.1
Copy link
Contributor

Choose a reason for hiding this comment

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

jsyk, you don't actually need to update the version number, the prerelease automation will do it automatically. No need to change it back though! the automation won't break if its already changed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I didn't know this! Very good to know ◡̈

Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
Copy link

SimpleCov Report

Coverage Threshold
Line 93.7% 93%
Branch 71.87% 71%

@hannahramadan hannahramadan merged commit 2d3dfec into dev Jan 25, 2024
28 checks passed
@hannahramadan hannahramadan deleted the view_component_regression_test branch April 12, 2024 19:28
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.

4 participants