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

Change default expressions to include start/end of line #150

Open
idontusenumbers opened this issue Aug 27, 2019 · 4 comments
Open

Change default expressions to include start/end of line #150

idontusenumbers opened this issue Aug 27, 2019 · 4 comments

Comments

@idontusenumbers
Copy link

Seems like disabling the 'match only first N' option works great if you change the default expressions to be more like:

^.*WARN.*$

instead of

.*WARN.*
@krasa
Copy link
Owner

krasa commented Oct 26, 2021

I do not understand the point of this

@idontusenumbers
Copy link
Author

idontusenumbers commented Oct 26, 2021

With the default expression, the match includes everything leading up to and then following the WARN message. This information is typically not part of the log entry.

@krasa
Copy link
Owner

krasa commented Oct 26, 2021

typically every line is tested separately, so both regexps would do the same

@idontusenumbers
Copy link
Author

That was not my experience.

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

No branches or pull requests

2 participants