Skip to content

Docs#5

Merged
IeuanWalker merged 5 commits intomasterfrom
docs
Jul 2, 2025
Merged

Docs#5
IeuanWalker merged 5 commits intomasterfrom
docs

Conversation

@IeuanWalker
Copy link
Owner

No description provided.

@IeuanWalker IeuanWalker requested a review from Copilot July 2, 2025 13:08
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

The PR refines how release notes outputs are handled, cleans up generated artifacts, and streamlines the work item fetch logic:

  • Separates markdown and HTML output parameters (and template parameters) in README and task.json with consistent naming and defaults.
  • Removes the committed output/release-notes.html generated file to prevent checking-in build artifacts.
  • Simplifies getWorkItem by removing the hard-coded fields list and using $expand=ALL.

Reviewed Changes

Copilot reviewed 4 out of 9 changed files in this pull request and generated 3 comments.

File Description
output/release-notes.html Deleted generated HTML output file
README.md Updated parameter names, defaults, and examples for markdown and HTML outputs
CommitRangeReleaseNotesTask/task/utils/WorkItemUtils.ts Removed unused fields array; now using $expand=ALL in the API call
CommitRangeReleaseNotesTask/task/task.json Changed default output paths to lowercase and split into markdown/html settings
Comments suppressed due to low confidence (2)

README.md:48

  • The table lists the .html default twice; outputFileMarkdown should default to a .md path.
| `outputFileMarkdown` and `outputFileHtml` | Path where generated release notes will be saved | ✅ | `$(Build.ArtifactStagingDirectory)/release-notes.html` and `$(Build.ArtifactStagingDirectory)/release-notes.html` |

@IeuanWalker IeuanWalker merged commit 80f417c into master Jul 2, 2025
1 check passed
@IeuanWalker IeuanWalker deleted the docs branch July 2, 2025 13:14
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.

1 participant