-
Notifications
You must be signed in to change notification settings - Fork 60
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
Combining multiple reports into a single GitHub issue #156
Comments
@zkoppert looking forward to your feedback on this. If you were considering to integrate a multi-report feature into the core functionality of issue-metrics itself, then some of the ideas above wouldn't be that useful any more. |
Yeah, these are great ideas!! I think that these are ready to implement as you described them here. I'll start by tackling the last one of always printing the title and search query: #157 |
Awesome! I will also try myself on one of the other ones. They sound like they shouldn't be too hard to implement but of course the devil is in the detail :) I think I will need with creating tests, as I wasn't clear from a first read of the repo what the testing approach is. So likely I will push the functionality with a minimal test, and then ask for review and help with further tests. |
This issue is stale because it has been open 21 days with no activity. Remove stale label or comment or this will be closed in 14 days. |
FYI here's an example of how we combine reports for Orchard Core. As you can see, not too complex, we just concat files in PowerShell. |
In InnerSourceCommons/InnerSourcePatterns#599 I learned that it is possible already to merge multiple reports into a single reports by apply a smart concatenation of multiple GHA steps. Very cool!
Here an example of how the resulting combined report looks like:
InnerSourceCommons/InnerSourcePatterns#601
I had some possible improvement ideas, related to this "combined reports" scenario:
Custom report title
The top-level title in the GitHub issue is always "Issue Metrics" right now. That is not really an issue if the GitHub issue only contains a single report, as the title of the GitHub issue itself can be used to provide a customized description of what the report is about.
However when a single GitHub issue contains multiple reports, this becomes more tricky.
Therefore we could add a configuration value
REPORT_TITLE
, roughly like this:REPORT_TITLE
Custom output file
The issue_metrics GHA always writes to the file
issue_metrics.md
.When combining multiple reports into a single issue it would be helpful to write to different filenames, so that those files can then be concatenated into a single file, which is then written to a GitHub issue.
OUTPUT_FILE
Always print the title and the search query
As shown in InnerSourceCommons/InnerSourcePatterns#601, an empty report will only say "no issues found for the given search criteria".
It would be helpful to always print the title, as well as the search query that was used.
That helps will debugging and identifying what the specific report was meant to be about.
Tasks
The text was updated successfully, but these errors were encountered: