Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@pamburus
Copy link
Owner

@pamburus pamburus commented Apr 12, 2025

This pull request makes fields.predefined.caller, fields.predefined.caller-file, and fields.predefined.caller-line options independent. They no longer override each other.

This may affect how the caller information is displayed if your configuration file has multiple combinations of these settings merged together.

If you get undesired results because of this change, it is recommended to review your configuration file and remove all unnecessary predefined field variants or split the configuration file into multiple files to make them unambiguous, then choose the appropriate one when viewing certain types of log files.

Closes #901

@codecov
Copy link

codecov bot commented Apr 12, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.83%. Comparing base (08891cd) to head (1cd906c).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #904      +/-   ##
==========================================
+ Coverage   82.71%   82.83%   +0.12%     
==========================================
  Files          46       46              
  Lines       12944    12971      +27     
==========================================
+ Hits        10706    10745      +39     
+ Misses       2238     2226      -12     

☔ 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.

@pamburus pamburus marked this pull request as ready for review April 12, 2025 18:57
@pamburus pamburus merged commit a904168 into master Apr 12, 2025
8 checks passed
@pamburus pamburus deleted the feature/caller branch April 12, 2025 19:11
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.

How to show caller along with caller-file and caller-line?

2 participants