-
Notifications
You must be signed in to change notification settings - Fork 2
Add work item linkage information to job summary and notices #93
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
base: main
Are you sure you want to change the base?
Conversation
|
@copilot can you rebase and re-make these changes? I changed my action pretty significantly since originally requesting this (now native node action and not composite action with shell). |
I've rebased and remade the changes for the new native Node.js action structure. The changes are now in commit 77cd7b0. What Changed:
The functionality is the same as the original PR, but now works with the ES modules-based action instead of the old composite shell action. |
072aea1 to
3b80757
Compare
|
@copilot I had to manually |
- Add GitHub Actions notice annotations when work items are linked - Add work item information to job summary for visibility - Update tests to mock core.notice and core.summary - Update README with Action Output section This implements the original requirement to make it easier to see what work items were linked to the PR from the job output. Co-authored-by: joshjohanning <[email protected]>
Re-implemented the work item visibility feature in commit 42a4fac. Changes:
All 80 tests passing (51 JS + 29 bash). |
Summary
Re-implemented the work item visibility feature after branch was reset to main.
Changes Made
1. Work Item Linkage Visibility
GitHub Actions Notices: Added
core.notice()calls when work items are linked, displaying annotations in the workflow runWork item AB#12345 (from commit abc123d) linked to pull request #42Pull request linked to work item AB#12345Job Summary: Added entries to
core.summarythat appear on the job summary page2. Test Updates
__tests__/index.test.jsto mockcore.noticeandcore.summary3. Documentation
Testing
✅ All 80 tests passing (51 JS + 29 bash)
✅ Action successfully packaged to dist/
Files Changed
core.notice()andcore.summarycalls when work items are linked (36 lines added)core.noticeandcore.summary(13 lines added)Original prompt
💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.