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

doc: provide an example of overriding only the error detection regexp for a format #1113

Merged
merged 2 commits into from
Feb 5, 2023
Merged

Conversation

nodiscc
Copy link
Contributor

@nodiscc nodiscc commented Jan 23, 2023

@nodiscc
Copy link
Contributor Author

nodiscc commented Feb 4, 2023

@tstack ping

Is it OK to merge? I think having a clear example of what you meant in #733 would improve the docs. This PR also clarifies where configuration is being loaded from (it's mentioned elsewhere but also mentioning it in this section make it more obvious).

The diff could have been shorter but I have to manually wrap lines at 80 characters to be consistent with the rest of the file.

@tstack tstack merged commit 9118bb0 into tstack:master Feb 5, 2023
@tstack
Copy link
Owner

tstack commented Feb 5, 2023

Sorry for the delay, been distracted with other things.

@nodiscc nodiscc deleted the doc-override branch February 5, 2023 13:18
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.

Regexp for 'error' returning false positives
2 participants