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

New Console output format (without tputs) #161

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

Conversation

sidz
Copy link
Collaborator

@sidz sidz commented Jul 6, 2022

Rework Console output in order to get rid of tpus.

That is how it looks like:
image

Fixes an issue #153

@sidz sidz requested a review from exussum12 July 6, 2022 20:11
return sprintf(
'%d) %s:%d Magic number: %s',
$index,
$detection->getFilePath(),
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Show real path instead of just filename as it is possible to have identical file names but under different namespaces.

Copy link
Collaborator

Choose a reason for hiding this comment

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

This changes the output format doesnt it ? May be better to wait for a new major version.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It may take ages to release new major :)

Copy link
Collaborator

Choose a reason for hiding this comment

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

Agreed but a break in a major version is not great either :(

@sidz sidz mentioned this pull request Jul 6, 2022
@sidz sidz mentioned this pull request Aug 18, 2022
@sidz sidz force-pushed the rework-console-output branch from 4fdeac9 to 4c218d8 Compare August 18, 2022 21:00
@sidz sidz force-pushed the rework-console-output branch from 4c218d8 to cc07c4d Compare August 18, 2022 21:09
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.

2 participants