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

use target instead of module_path in formatter by default #205

Closed
wants to merge 1 commit into from

Conversation

xofyarg
Copy link

@xofyarg xofyarg commented May 6, 2021

Target is used in filter, and its value defaults to module_path. To
enable user customize the format, also to reduce confusion, makes
formatter supporting target as well.

This change introduces a boolean builder variable "honor_target". When
set to true(the default), formatter will use target in the record
metadata.

Fix #155 and #179.

Target is used in filter, and its value defaults to module_path. To
enable user customize the format, also to reduce confusion, makes
formatter supporting target as well.

This change introduces a boolean builder variable "honor_target". When
set to true(the default), formatter will use target in the record
metadata.

Fix rust-cli#155 and rust-cli#179.
@deastoe
Copy link

deastoe commented Jun 25, 2021

I'd love to have this feature, even if the default behaviour remains the same.

@mainrs
Copy link
Contributor

mainrs commented Jul 14, 2021

Superseeded by #209. Thanks for opening a PR and sorry that I wasn't active to merge it earlier on

@mainrs mainrs closed this Jul 14, 2021
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.

Module path and target confusion
3 participants