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

pattern-metrics: Merge to one report #599

Merged
merged 3 commits into from
Oct 25, 2023
Merged

pattern-metrics: Merge to one report #599

merged 3 commits into from
Oct 25, 2023

Conversation

zkoppert
Copy link
Member

Works on one aspect of #565 by merging the 2 issues generated by this metrics workflow into a single issue.

@spier Let me know what you think of this approach.

The issue generated will look like this:

< Report on issues content >


< Report on pull requests content>

Copy link
Member

@spier spier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great.

Making some inline changes, and will merge and test this shortly.

.github/workflows/pattern-metrics.yaml Outdated Show resolved Hide resolved
.github/workflows/pattern-metrics.yaml Show resolved Hide resolved
@spier spier changed the title Merge to one report pattern-metrics: Merge to one report Oct 25, 2023
@spier spier merged commit 9942807 into InnerSourceCommons:main Oct 25, 2023
2 of 6 checks passed
@spier
Copy link
Member

spier commented Oct 25, 2023

Thank you for this contribution @zkoppert !

I made some fixes to the YAML, and now it all works.
Note to self: Don't make un-tested YAML changes directly via the UI. Too easy to mess up indentation :)

Now the created issue with the combined reports looks like this:
#601

Based on how this merging of the reports into a single reports works, I had some feature ideas for the issue-metrics GHA itself. Will create a new issue for this.

rmarting pushed a commit to rmarting/InnerSourcePatterns that referenced this pull request Feb 29, 2024
* Merge the 2 pattern-metrics reports into a single report
* Change title of created issue
* Add 'community metrics' label to created issue

---------

Co-authored-by: Sebastian Spier <github@spier.hu>
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.

2 participants