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

Print mutation details on inlined code detection warning #685

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Vampire
Copy link
Contributor

@Vampire Vampire commented Oct 1, 2019

No description provided.

@Vampire Vampire force-pushed the print-mutation-details-on-inlined-code-detection-warning branch from 9b8306a to d572600 Compare October 5, 2019 12:36
@Vampire Vampire force-pushed the print-mutation-details-on-inlined-code-detection-warning branch from d572600 to 3ddf829 Compare May 8, 2023 03:05
@Vampire
Copy link
Contributor Author

Vampire commented May 8, 2023

I updated this PR to latest master, maybe you have time for it now. :-)

@Vampire Vampire force-pushed the print-mutation-details-on-inlined-code-detection-warning branch 4 times, most recently from 8e2e43a to 9c35e4c Compare May 11, 2023 11:17
@hcoles
Copy link
Owner

hcoles commented May 11, 2023

This will make the log even more noisy. There isn't really an action the user can take when this occurs (beyound raising an issue) so it was probably incorrect to make it a warning in the first place. If you can update it to be a debug message I'll merge it in.

@Vampire
Copy link
Contributor Author

Vampire commented May 11, 2023

I don't fully remember as it is 3 years ago.
But back then I probably greatly missed this information and needed it to properly process without much debugging.
Also, this additional info is on a warning.
Warnings are hopefully not happen that often and if they happen, the average developer should try to get rid of them and welcome to have the necessary information for that. :-)

Ok, that was a lie, the average developer ignores warnings, but don't destroy my dreams. :-D

@Vampire Vampire force-pushed the print-mutation-details-on-inlined-code-detection-warning branch 6 times, most recently from d5c0960 to d1d6258 Compare May 19, 2023 13:23
@Vampire
Copy link
Contributor Author

Vampire commented May 24, 2023

Yeah, it's still the same.

13:10:23 PIT >> WARNING : Found more than one mutation similar on same line in a finally block. Can't correct for inlining.
13:10:23 PIT >> WARNING : Found more than one mutation similar on same line in a finally block. Can't correct for inlining.
13:10:23 PIT >> WARNING : Found more than one mutation similar on same line in a finally block. Can't correct for inlining.
13:10:23 PIT >> WARNING : Found more than one mutation similar on same line in a finally block. Can't correct for inlining.

with no way to see what might be the problem to at least be able to report it properly.

@Vampire Vampire force-pushed the print-mutation-details-on-inlined-code-detection-warning branch from d1d6258 to 093e47b Compare May 24, 2023 11:18
@Vampire
Copy link
Contributor Author

Vampire commented May 24, 2023

If you are concerned about the noise, how about adding the details only in VERBOSE and VERBOSE_NO_SPINNER?

@Vampire Vampire force-pushed the print-mutation-details-on-inlined-code-detection-warning branch 2 times, most recently from a40d70c to 9650585 Compare May 30, 2023 09:20
@Vampire Vampire force-pushed the print-mutation-details-on-inlined-code-detection-warning branch from 9650585 to 83b787e Compare June 10, 2023 19:33
@Vampire Vampire force-pushed the print-mutation-details-on-inlined-code-detection-warning branch from 83b787e to fa0b629 Compare June 22, 2023 19:49
@Vampire Vampire force-pushed the print-mutation-details-on-inlined-code-detection-warning branch from fa0b629 to d227701 Compare June 23, 2023 12:58
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.

None yet

2 participants