Skip to content

DocumentationMonster-0.4.4.0#337338

Open
RickStrahl wants to merge 1 commit intomicrosoft:masterfrom
RickStrahl:DocumentationMonster-0.4.4.0
Open

DocumentationMonster-0.4.4.0#337338
RickStrahl wants to merge 1 commit intomicrosoft:masterfrom
RickStrahl:DocumentationMonster-0.4.4.0

Conversation

@RickStrahl
Copy link
Contributor

@RickStrahl RickStrahl commented Feb 7, 2026

Checklist for Pull Requests

Manifests

  • Have you checked that there aren't other open pull requests for the same manifest update/change?
  • This PR only modifies one (1) manifest
  • Have you validated your manifest locally with winget validate --manifest <path>?
  • Have you tested your manifest locally with winget install --manifest <path>?
  • Does your manifest conform to the 1.10 schema?

Note: <path> is the directory's name containing the manifest you're submitting.


Microsoft Reviewers: Open in CodeFlow

Copilot AI review requested due to automatic review settings February 7, 2026 07:48
@wingetbot
Copy link
Collaborator

Validation Pipeline Run WinGetSvc-Validation-132-337338-20260207-1

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

Adds a new WinGet manifest set for WestWind.DocumentationMonster version 0.4.4.0 under the community repository’s multi-file manifest structure (version/defaultLocale/installer), enabling install/upgrade via winget.

Changes:

  • Added the version manifest declaring identifier, version, and default locale.
  • Added the defaultLocale manifest with publisher/package metadata and documentation links.
  • Added the installer manifest for the Inno Setup installer, including download URL, SHA256, commands, and release date.

Reviewed changes

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

File Description
manifests/w/WestWind/DocumentationMonster/0.4.4.0/WestWind.DocumentationMonster.yaml New version manifest for 0.4.4.0
manifests/w/WestWind/DocumentationMonster/0.4.4.0/WestWind.DocumentationMonster.locale.en-US.yaml New en-US defaultLocale metadata for 0.4.4.0
manifests/w/WestWind/DocumentationMonster/0.4.4.0/WestWind.DocumentationMonster.installer.yaml New installer definition (Inno) for 0.4.4.0

- Windows.Desktop
InstallerType: inno
InstallModes:
- silent
Copy link

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

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

InstallModes lists only silent, but the schema guidance for the community repo requires installers to support both silent and silentWithProgress. Add silentWithProgress here (and ensure the Inno installer supports it via default behavior or explicit switches).

Suggested change
- silent
- silent
- silentWithProgress

Copilot uses AI. Check for mistakes.
ElevationRequirement: elevatesSelf
Installers:
- Architecture: neutral
InstallerUrl: https://raw-githubusercontent-com-gh.computerqwq.top/RickStrahl/DocumentationMonster/refs/heads/main/Releases/v1/v0.4/DocumentationMonsterSetup-0.4.4.0.exe
Copy link

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

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

InstallerUrl points at raw.githubusercontent.com/.../refs/heads/main/..., which is mutable and can change without a version bump. That can break installs (hash mismatch) and reduces supply-chain reliability. Prefer an immutable, versioned download URL (e.g., a tagged GitHub Release asset or vendor-hosted versioned URL).

Suggested change
InstallerUrl: https://raw.githubusercontent.com/RickStrahl/DocumentationMonster/refs/heads/main/Releases/v1/v0.4/DocumentationMonsterSetup-0.4.4.0.exe
InstallerUrl: https://github.com/RickStrahl/DocumentationMonster/releases/download/v0.4.4.0/DocumentationMonsterSetup-0.4.4.0.exe

Copilot uses AI. Check for mistakes.
@wingetbot wingetbot added New-Manifest Azure-Pipeline-Passed Validation pipeline passed. There may still be manual validation requirements. Validation-Completed Validation passed labels Feb 7, 2026
@stephengillie stephengillie added the Moderator-Approved One of the Moderators has reviewed and approved this PR label Feb 7, 2026
@stephengillie
Copy link
Collaborator

Merging failed with:

Error:
{
"message": "All comments must be resolved.",
"documentation_url": "https://docs.github.com/articles/about-protected-branches",
"status": "405"
} - Url - Body: {"merge_method":"squash","sha":"7b2bcab9973fce89f36c34115fbc25a14431605a"}

(Automated response - build 1213.)

@stephengillie
Copy link
Collaborator

Merging failed with:

Error:
{
"message": "All comments must be resolved.",
"documentation_url": "https://docs.github.com/articles/about-protected-branches",
"status": "405"
} - Url - Body: {"merge_method":"squash","sha":"7b2bcab9973fce89f36c34115fbc25a14431605a"}

(Automated response - build 1213.)

@stephengillie
Copy link
Collaborator

Merging failed with:

Error:
{
"message": "All comments must be resolved.",
"documentation_url": "https://docs.github.com/articles/about-protected-branches",
"status": "405"
} - Url - Body: {"merge_method":"squash","sha":"7b2bcab9973fce89f36c34115fbc25a14431605a"}

(Automated response - build 1213.)

@stephengillie
Copy link
Collaborator

Merging failed with:

Error:
{
"message": "All comments must be resolved.",
"documentation_url": "https://docs.github.com/articles/about-protected-branches",
"status": "405"
} - Url - Body: {"merge_method":"squash","sha":"7b2bcab9973fce89f36c34115fbc25a14431605a"}

(Automated response - build 1213.)

@stephengillie
Copy link
Collaborator

Merging failed with:

Error:
{
"message": "All comments must be resolved.",
"documentation_url": "https://docs.github.com/articles/about-protected-branches",
"status": "405"
} - Url - Body: {"merge_method":"squash","sha":"7b2bcab9973fce89f36c34115fbc25a14431605a"}

(Automated response - build 1213.)

@stephengillie
Copy link
Collaborator

Merging failed with:

Error:
{
"message": "All comments must be resolved.",
"documentation_url": "https://docs.github.com/articles/about-protected-branches",
"status": "405"
} - Url - Body: {"merge_method":"squash","sha":"7b2bcab9973fce89f36c34115fbc25a14431605a"}

(Automated response - build 1213.)

@stephengillie
Copy link
Collaborator

Merging failed with:

Error:
{
"message": "All comments must be resolved.",
"documentation_url": "https://docs.github.com/articles/about-protected-branches",
"status": "405"
} - Url - Body: {"merge_method":"squash","sha":"7b2bcab9973fce89f36c34115fbc25a14431605a"}

(Automated response - build 1213.)

@stephengillie
Copy link
Collaborator

Merging failed with:

Error:
{
"message": "All comments must be resolved.",
"documentation_url": "https://docs.github.com/articles/about-protected-branches",
"status": "405"
} - Url - Body: {"merge_method":"squash","sha":"7b2bcab9973fce89f36c34115fbc25a14431605a"}

(Automated response - build 1213.)

@stephengillie
Copy link
Collaborator

Merging failed with:

Error:
{
"message": "All comments must be resolved.",
"documentation_url": "https://docs.github.com/articles/about-protected-branches",
"status": "405"
} - Url - Body: {"merge_method":"squash","sha":"7b2bcab9973fce89f36c34115fbc25a14431605a"}

(Automated response - build 1213.)

@stephengillie
Copy link
Collaborator

Merging failed with:

Error:
{
"message": "All comments must be resolved.",
"documentation_url": "https://docs.github.com/articles/about-protected-branches",
"status": "405"
} - Url - Body: {"merge_method":"squash","sha":"7b2bcab9973fce89f36c34115fbc25a14431605a"}

(Automated response - build 1213.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Azure-Pipeline-Passed Validation pipeline passed. There may still be manual validation requirements. Moderator-Approved One of the Moderators has reviewed and approved this PR New-Manifest Validation-Completed Validation passed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants