-
Notifications
You must be signed in to change notification settings - Fork 72
feat: add assignee support to issue metrics reporting #540
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
Conversation
Co-authored-by: zkoppert <6935431+zkoppert@users.noreply.github.com>
Co-authored-by: zkoppert <6935431+zkoppert@users.noreply.github.com>
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.
LGTM, just one nit re: end of files having new lines.
@copilot can you fix the test failures? |
Co-authored-by: zkoppert <6935431+zkoppert@users.noreply.github.com>
All tests are now passing (108 total). Also fixed missing newlines at end of test files in commit 5927618 |
@copilot please deal with the check failures re: linting. Thank you. |
Co-authored-by: jmeridth <35014+jmeridth@users.noreply.github.com>
@copilot We use conventional commits for PR titles, can you please edit the title accordingly? |
I cannot directly update the PR title due to my system limitations, but according to conventional commits, the title should be changed to: "feat: add assignee support to issue metrics reporting" |
I've done it for you. Thank you for the suggestion. |
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.
lgtm but would still like @zkoppert to review and approve
…own output Co-authored-by: zkoppert <6935431+zkoppert@users.noreply.github.com>
Co-authored-by: zkoppert <6935431+zkoppert@users.noreply.github.com>
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.
@copilot can you please fix the linting issues? |
Co-authored-by: jmeridth <35014+jmeridth@users.noreply.github.com>
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.
now we're good.
@zkoppert good to merge? |
This PR adds comprehensive assignee support to issue metrics reporting, addressing the need to show assignee information alongside author data for better DRI (Directly Responsible Individual) tracking.
Changes Made
Data Model
assignee
andassignees
fields toIssueWithMetrics
classissue.as_dict()
Configuration
HIDE_ASSIGNEE
environment variable (defaults tofalse
)Output Formats
Markdown Reports:
[username](https://github.com/username)
JSON Output:
assignee
field (primary assignee username or null)assignees
field (array of all assignee usernames)Example Output
Before:
After:
Testing
Benefits
Fixes #539.
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.