-
Notifications
You must be signed in to change notification settings - Fork 186
Add functionality to link artifacts to work items #377
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
Add functionality to link artifacts to work items #377
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #377 +/- ##
=======================================
Coverage ? 91.52%
=======================================
Files ? 13
Lines ? 838
Branches ? 142
=======================================
Hits ? 767
Misses ? 40
Partials ? 31
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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.
My main comment on this is that the user is going to need to know what the format of the artifact link is. And of course, each type is different. I don't know if there is a good way around this or not. I was hoping we could send the type and then some identifiers to build the uri accordingly. I am okay with giving this a try to start to see how to goes.
…t types and URI construction options
Adds a new `wit_add_artifact_link` tool to enable linking work items to repository artifacts (`branches`, `commits`, `pull requests`). ## 🔧 Core Implementation Added `wit_add_artifact_link` tool in [workitems.ts](https://vscode.dev/github/microsoft/azure-devops-mcp/blob/main/src/tools/workitems.ts) ## 📚 Documentation Updated [EXAMPLES.md](https://vscode.dev/github/microsoft/azure-devops-mcp/blob/main/docs/EXAMPLES.md) with new "Adding Artifact Links" section ## ✅ Testing Added test coverage in [workitems.test.ts](https://vscode.dev/github/microsoft/azure-devops-mcp/blob/main/test/src/tools/workitems.test.ts) ## GitHub issue number microsoft#279 ## **Associated Risks** None ## ✅ **PR Checklist** - [x] **I have read the [contribution guidelines](https://github.com/microsoft/azure-devops-mcp/blob/main/CONTRIBUTING.md)** - [x] **I have read the [code of conduct guidelines](https://github.com/microsoft/azure-devops-mcp/blob/main/CODE_OF_CONDUCT.md)** - [x] Title of the pull request is clear and informative. - [x] 👌 Code hygiene - [x] 🔭 Telemetry added, updated, or N/A - [x] 📄 Documentation added, updated, or N/A - [x] 🛡️ Automated tests added, or N/A ## 🧪 **How did you test it?** Run `npm run test` , `npm run build` , `npm run validate-tools` --------- Co-authored-by: Dan Hellem <[email protected]>
Adds a new
wit_add_artifact_link
tool to enable linking work items to repository artifacts (branches
,commits
,pull requests
).🔧 Core Implementation
Added
wit_add_artifact_link
tool in workitems.ts📚 Documentation
Updated EXAMPLES.md with new "Adding Artifact Links" section
✅ Testing
Added test coverage in workitems.test.ts
GitHub issue number
#279
Associated Risks
None
✅ PR Checklist
🧪 How did you test it?
Run
npm run test
,npm run build
,npm run validate-tools