Skip to content

Mosquitto test fixes - #1535

Merged
aaronpowell merged 2 commits into
mainfrom
fixing-mosquitto-image
Aug 21, 2026
Merged

Mosquitto test fixes#1535
aaronpowell merged 2 commits into
mainfrom
fixing-mosquitto-image

Conversation

@aaronpowell

Copy link
Copy Markdown
Member

Closes #<ISSUE_NUMBER>

PR Checklist

  • Created a feature/dev branch in your fork (vs. submitting directly from a commit on main)
  • Based off latest main branch of toolkit
  • PR doesn't include merge commits (always rebase on top of our main, if needed)
  • New integration
    • Docs are written
    • Added description of major feature to project description for NuGet package (4000 total character limit, so don't push entire description over that)
  • Tests for the changes have been added (for bug fixes / features) (if applicable)
  • Contains NO breaking changes
  • Every new API (including internal ones) has full XML docs
  • Code follows all style conventions

Other information

Copilot AI lite review requested due to automatic review settings August 20, 2026 06:08

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the Mosquitto hosting integration’s pinned container image reference. This sits in the Mosquitto integration’s container image tag constants, which are used by the builder extensions to configure the container resource and are validated by the Mosquitto test suite.

Changes:

  • Changes the Docker image name from eclipse-mosquitto to library/eclipse-mosquitto.
Show a summary per file
File Description
src/CommunityToolkit.Aspire.Hosting.Mosquitto/MosquittoContainerImageTags.cs Adjusts the Docker Hub image name used by the Mosquitto integration.

Review details

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Lite

Comment on lines 8 to 10
/// <remarks>eclipse-mosquitto</remarks>
internal const string Image = "eclipse-mosquitto";
internal const string Image = "library/eclipse-mosquitto";

@github-actions

Copy link
Copy Markdown
Contributor

Code Coverage

Package Line Rate Branch Rate Complexity Health
CommunityToolkit.Aspire.Hosting.Mosquitto 90% 67% 17
Summary 90% (55 / 61) 67% (8 / 12) 17

@aaronpowell
aaronpowell deployed to azure-artifacts August 20, 2026 23:44 — with GitHub Actions Active
@aaronpowell
aaronpowell merged commit 437177d into main Aug 21, 2026
16 checks passed
@aaronpowell
aaronpowell deleted the fixing-mosquitto-image branch August 21, 2026 01:36
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