-
-
Notifications
You must be signed in to change notification settings - Fork 379
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
feat(ChangesReporting): add GitHub output formatter #6747
feat(ChangesReporting): add GitHub output formatter #6747
Conversation
9053336
to
7185c7d
Compare
@@ -16,15 +16,17 @@ | |||
/** | |||
* @var string | |||
* @see https://en.wikipedia.org/wiki/Diff#Unified_format | |||
* @see https://regex101.com/r/AUPIX4/1 | |||
* @see https://regex101.com/r/sohQIG/1 |
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.
use increase number over generate new one, you can click "Update Regex" instead of "Save new Regex"
just on final change when this PR is ready so only change /1 to /2 so we have history on regex1.com on what the different with previous approach
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.
Ok no problem, I didn't want change the original one by myself (at least without your permission)
a4e705e
to
a5f2745
Compare
Signed-off-by: Emilien Escalle <[email protected]>
34b0f04
to
b92a1d7
Compare
@samsonasik do you need anything else for this PR? Let me know. |
I am ok on this, I will let @TomasVotruba decide for merging |
Thank you 👍 Let's try this out |
is this auto-enabled when in github CI or do we need to manualy enable this via CLI flag? and if so, can you provide an example for the CLI flag to enable this.... thanks |
oh nevermind i see.. it's |
Add a new reporting output formatter for GitHub:
Example:
Summary annotations
PR view