Skip to content

Add support for watcher annotations in the Files tab - #248

Merged
alejandro5042 merged 2 commits into
alejandro5042:masterfrom
kroeschl:watchers-annotation
Jun 19, 2026
Merged

alejandro5042 merged 2 commits into
alejandro5042:masterfrom
kroeschl:watchers-annotation

Conversation

@kroeschl

@kroeschl kroeschl commented May 20, 2026

Copy link
Copy Markdown
Contributor

Justification

I was looking at the the Files tab on a PR recently to see why I'd been added to the PR by our owners tooling, and didn't see any annotated files. The Duties tab showed that I'm a watch for one file, but it was annotated as expect. That's because the Owners-Watcher role didn't exist (I think) when owner, alternate, and reviewer/expert file annotations were added in this script.
image

Implementation

Using the implementation for expert/reviewer as an example, add watcher annotations.

Testing

Tested against the same PR, and I now see the annotation. Note that the annotation and expander overlap is an existing issue with all Files annotations that I'm not addressing in this PR.
image
Also verified that the notification content renders as expected.

@kroeschl
kroeschl marked this pull request as ready for review May 20, 2026 18:45
@kroeschl
kroeschl force-pushed the watchers-annotation branch from b841ae6 to e5592ff Compare June 19, 2026 15:11
@kroeschl
kroeschl force-pushed the watchers-annotation branch from e5592ff to e6aab8c Compare June 19, 2026 15:12

@alejandro5042 alejandro5042 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thank you!

@alejandro5042
alejandro5042 merged commit fc734e8 into alejandro5042:master Jun 19, 2026
3 checks passed
@kroeschl
kroeschl deleted the watchers-annotation branch June 19, 2026 19:25
kroeschl added a commit to kroeschl/azdo-userscripts that referenced this pull request Jul 15, 2026
* Annotate watchers in the Files tab

* Update version and update notification
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