-
Notifications
You must be signed in to change notification settings - Fork 89
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>
jmeridth
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.
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. |
jmeridth
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.
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>
jmeridth
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.
|
@copilot can you please fix the linting issues? |
Co-authored-by: jmeridth <35014+jmeridth@users.noreply.github.com>
jmeridth
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.
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
assigneeandassigneesfields toIssueWithMetricsclassissue.as_dict()Configuration
HIDE_ASSIGNEEenvironment variable (defaults tofalse)Output Formats
Markdown Reports:
[username](https://github.com/username)JSON Output:
assigneefield (primary assignee username or null)assigneesfield (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.