Skip to content

Windows FMA - Mattermost - #46479

Merged
allenhouchins merged 2 commits into
mainfrom
win-fma-mattermost
May 29, 2026
Merged

Windows FMA - Mattermost#46479
allenhouchins merged 2 commits into
mainfrom
win-fma-mattermost

Conversation

@harrisonravazzolo

@harrisonravazzolo harrisonravazzolo commented May 29, 2026

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • New Features
    • Added Windows platform support for Mattermost v6.0.4, including MSI installer with automated deployment and uninstall capabilities, integrity verification, and upgrade management.

Review Change Stack

@harrisonravazzolo
harrisonravazzolo marked this pull request as ready for review May 29, 2026 19:19

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.

Tip: disable this comment in your organization's Code Review settings.

@coderabbitai

coderabbitai Bot commented May 29, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 37fd0385-71fc-4eea-861e-278c9ed4518b

📥 Commits

Reviewing files that changed from the base of the PR and between 032246d and 938e50b.

📒 Files selected for processing (3)
  • ee/maintained-apps/inputs/winget/mattermost.json
  • ee/maintained-apps/outputs/apps.json
  • ee/maintained-apps/outputs/mattermost/windows.json

Walkthrough

This PR adds Mattermost Windows desktop application support to Fleet's maintained apps system. It introduces a Winget input manifest defining Mattermost metadata (identifier, x64 architecture, MSI installer, machine-wide scope), registers the application in the apps catalog as mattermost/windows, and provides Windows-specific configuration including version 6.0.4 detection logic, MSI installer details with SHA256 integrity verification, and PowerShell scripts for automated installation via msiexec and uninstallation via upgrade code.

Possibly related PRs

  • fleetdm/fleet#46425: Adds another Windows FMA app definition using the same pattern: Winget input manifest, apps.json registration, and windows.json platform configuration with MSI detection and PowerShell install/uninstall script wiring.
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning No pull request description was provided by the author, but the template requires multiple sections including a related issue, checklist items for validation, testing, database changes, and configuration. Add a complete pull request description following the template, including the related issue number, applicable checklist items, and testing confirmation.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Windows FMA - Mattermost' is specific and clearly indicates the main change: adding Windows FMA (Fleet Managed Apps) support for Mattermost.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch win-fma-mattermost

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@allenhouchins
allenhouchins merged commit fbfcf5f into main May 29, 2026
13 checks passed
@allenhouchins
allenhouchins deleted the win-fma-mattermost branch May 29, 2026 21:57
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