-
Notifications
You must be signed in to change notification settings - Fork 5
feat: add new formatter log output for github #59
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
base: main
Are you sure you want to change the base?
Conversation
c343cf2 to
f2418cf
Compare
9eaefbd to
9ea579c
Compare
8891875 to
932d12b
Compare
|
@theseion Looks like test is failing (cmd/regex_compare_test.go:151) because the error was not triggered per |
932d12b to
58bbb52
Compare
theseion
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
❤️
Signed-off-by: Felipe Zipitria <felipe.zipitria@owasp.org>
Signed-off-by: Felipe Zipitria <felipe.zipitria@owasp.org>
eda978b to
7422dfa
Compare
Signed-off-by: Felipe Zipitria <felipe.zipitria@owasp.org>
7422dfa to
43331f6
Compare
| text: "this is a warning message", | ||
| logType: zerolog.WarnLevel, | ||
| want: "::warn ::this is a warning message", | ||
| }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add a test for error.
Signed-off-by: Felipe Zipitria felipe.zipitria@owasp.org
Modify logger to output in a format parseable by Github's runners.
Fixes #56.
Pending: