Skip to content

#1617 avoid useless NoSuchMethodRuntimeException in logs#1620

Merged
asolntsev merged 1 commit intomainfrom
fix/1617-avoid-useless-exception-in-logs
Aug 20, 2025
Merged

#1617 avoid useless NoSuchMethodRuntimeException in logs#1620
asolntsev merged 1 commit intomainfrom
fix/1617-avoid-useless-exception-in-logs

Conversation

@asolntsev
Copy link
Copy Markdown
Collaborator

No description provided.

@asolntsev asolntsev added this to the 2.4.5 milestone Aug 20, 2025
@asolntsev asolntsev self-assigned this Aug 20, 2025
@asolntsev asolntsev added the bug Something isn't working label Aug 20, 2025
@what-the-diff
Copy link
Copy Markdown

what-the-diff bot commented Aug 20, 2025

PR Summary

  • Logging Level Adjustment for Specific Exception
    This update enhances our system's logging feature by including a new condition. Now, if a specific type of error called NoSuchMethodRuntimeException occurs, the system's logging level will change accordingly. This means we'll receive more detailed information whenever such an error happens, enabling quicker and more accurate troubleshooting.

@codecov-commenter
Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 92.33%. Comparing base (eb179f2) to head (3292cec).

Files with missing lines Patch % Lines
.../java/net/datafaker/service/FakeValuesService.java 0.00% 0 Missing and 1 partial ⚠️
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #1620      +/-   ##
============================================
- Coverage     92.41%   92.33%   -0.08%     
+ Complexity     3357     3354       -3     
============================================
  Files           331      331              
  Lines          6618     6618              
  Branches        655      655              
============================================
- Hits           6116     6111       -5     
- Misses          346      348       +2     
- Partials        156      159       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@bodiam
Copy link
Copy Markdown
Contributor

bodiam commented Aug 20, 2025

LGTM, feel free to merge this. Thanks for fixing this so quickly!

@asolntsev asolntsev merged commit 32ae4d8 into main Aug 20, 2025
13 checks passed
@asolntsev asolntsev deleted the fix/1617-avoid-useless-exception-in-logs branch August 20, 2025 11:10
@asolntsev
Copy link
Copy Markdown
Collaborator Author

Seems we should release 2.4.5 soon?

@bodiam
Copy link
Copy Markdown
Contributor

bodiam commented Aug 20, 2025

Seems we should release 2.4.5 soon?

I'm happy if we release a new version. You should be able to make the release, so if you want to, feel free, I have no issue with that, but perhaps we can get those 2 open PRs in?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

java.lang.NoSuchMethodException is logged on Text.text expression with length parameter

3 participants