Skip to content

Conversation

@mguinness
Copy link

Related #940

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

Copilot AI review requested due to automatic review settings November 24, 2025 03:09
Copilot finished reviewing on behalf of mguinness November 24, 2025 03:11
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

This PR adds the Papercut SMTP hosting integration to the list of available packages in the README.md, making it discoverable to users of the Aspire Community Toolkit.

Key changes:

  • Added Papercut SMTP integration entry to the integrations table with links to documentation and NuGet packages
  • Added markdown reference link definitions for the integration's documentation and NuGet package URLs

[mailpit-ext-nuget-preview]: https://nuget.org/packages/CommunityToolkit.Aspire.Hosting.MailPit/absoluteLatest
[papercutsmtp-integration-docs]: https://github.com/CommunityToolkit/Aspire/tree/main/src/CommunityToolkit.Aspire.Hosting.PapercutSmtp#readme
[papercutsmtp-shields]: https://img.shields.io/nuget/v/CommunityToolkit.Aspire.Hosting.PapercutSmtp
[papercutsmtp-nuget]: https://www.nuget.org/packages/CommunityToolkit.Aspire.Hosting.PapercutSmtp/
Copy link

Copilot AI Nov 24, 2025

Choose a reason for hiding this comment

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

The NuGet URL uses www.nuget.org while all other integrations consistently use nuget.org (without www). For consistency with the rest of the file, change to:

[papercutsmtp-nuget]: https://nuget.org/packages/CommunityToolkit.Aspire.Hosting.PapercutSmtp/
Suggested change
[papercutsmtp-nuget]: https://www.nuget.org/packages/CommunityToolkit.Aspire.Hosting.PapercutSmtp/
[papercutsmtp-nuget]: https://nuget.org/packages/CommunityToolkit.Aspire.Hosting.PapercutSmtp/

Copilot uses AI. Check for mistakes.
| - **Learn More**: [`Hosting.SqlServer.Extensions`][sqlserver-ext-integration-docs] <br /> - Stable 📦: [![CommunityToolkit.Aspire.SqlServer.Extensions][sqlserver-ext-shields]][sqlserver-ext-nuget] <br /> - Preview 📦: [![CommunityToolkit.Aspire.Hosting.SqlServer.Extensions][sqlserver-ext-shields-preview]][sqlserver-ext-nuget-preview] | An integration that contains some additional extensions for hosting SqlServer container. |
| - **Learn More**: [`Hosting.LavinMQ`][lavinmq-integration-docs] <br /> - Stable 📦: [![CommunityToolkit.Aspire.Hosting.LavinMQ][lavinmq-shields]][lavinmq-nuget] <br /> - Preview 📦: [![CommunityToolkit.Aspire.Hosting.LavinMQ][lavinmq-shields-preview]][lavinmq-nuget-preview] | An Aspire hosting integration for [LavinMQ](https://www.lavinmq.com). |
| - **Learn More**: [`Hosting.MailPit`][mailpit-ext-integration-docs] <br /> - Stable 📦: [![CommunityToolkit.Aspire.Hosting.MailPit][mailpit-ext-shields]][mailpit-ext-nuget] <br /> - Preview 📦: [![CommunityToolkit.Aspire.Hosting.MailPit][mailpit-ext-shields-preview]][mailpit-ext-nuget-preview] | An Aspire integration leveraging the [MailPit](https://mailpit.axllent.org/) container. |
| - **Learn More**: [`Hosting.PapercutSmtp`][papercutsmtp-integration-docs] <br /> - Stable 📦: [![CommunityToolkit.Aspire.Hosting.PapercutSmtp][papercutsmtp-shields]][papercutsmtp-nuget] <br /> - Preview 📦: [![CommunityToolkit.Aspire.Hosting.PapercutSmtp][papercutsmtp-shields-preview]][papercutsmtp-nuget-preview] | An Aspire integration leveraging the [Papercut SMTP](https://github.com/ChangemakerStudios/Papercut-SMTP) container. |
Copy link

Copilot AI Nov 24, 2025

Choose a reason for hiding this comment

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

This line has excessive trailing whitespace before the pipe character (|) separating the first and second columns. The spacing should match other entries in the table (approximately 23 spaces based on line 48 and 50). The current line appears to have significantly more spacing, breaking visual alignment in the markdown table.

Suggested change
| - **Learn More**: [`Hosting.PapercutSmtp`][papercutsmtp-integration-docs] <br /> - Stable 📦: [![CommunityToolkit.Aspire.Hosting.PapercutSmtp][papercutsmtp-shields]][papercutsmtp-nuget] <br /> - Preview 📦: [![CommunityToolkit.Aspire.Hosting.PapercutSmtp][papercutsmtp-shields-preview]][papercutsmtp-nuget-preview] | An Aspire integration leveraging the [Papercut SMTP](https://github.com/ChangemakerStudios/Papercut-SMTP) container. |
| - **Learn More**: [`Hosting.PapercutSmtp`][papercutsmtp-integration-docs] <br /> - Stable 📦: [![CommunityToolkit.Aspire.Hosting.PapercutSmtp][papercutsmtp-shields]][papercutsmtp-nuget] <br /> - Preview 📦: [![CommunityToolkit.Aspire.Hosting.PapercutSmtp][papercutsmtp-shields-preview]][papercutsmtp-nuget-preview] | An Aspire integration leveraging the [Papercut SMTP](https://github.com/ChangemakerStudios/Papercut-SMTP) container. |

Copilot uses AI. Check for mistakes.
[mailpit-ext-nuget]: https://nuget.org/packages/CommunityToolkit.Aspire.Hosting.MailPit/
[mailpit-ext-shields-preview]: https://img.shields.io/nuget/vpre/CommunityToolkit.Aspire.Hosting.MailPit?label=nuget%20(preview)
[mailpit-ext-nuget-preview]: https://nuget.org/packages/CommunityToolkit.Aspire.Hosting.MailPit/absoluteLatest
[papercutsmtp-integration-docs]: https://github.com/CommunityToolkit/Aspire/tree/main/src/CommunityToolkit.Aspire.Hosting.PapercutSmtp#readme
Copy link

Copilot AI Nov 24, 2025

Choose a reason for hiding this comment

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

The documentation link uses a GitHub URL pointing to the source directory, while other integrations (like LavinMQ, MailPit, k6) use Microsoft Learn documentation URLs (e.g., https://learn.microsoft.com/dotnet/aspire/community-toolkit/hosting-<integration>). Consider using a consistent pattern:

[papercutsmtp-integration-docs]: https://learn.microsoft.com/dotnet/aspire/community-toolkit/hosting-papercutsmtp

This would be consistent with the MailPit integration above and other hosting integrations.

Suggested change
[papercutsmtp-integration-docs]: https://github.com/CommunityToolkit/Aspire/tree/main/src/CommunityToolkit.Aspire.Hosting.PapercutSmtp#readme
[papercutsmtp-integration-docs]: https://learn.microsoft.com/dotnet/aspire/community-toolkit/hosting-papercutsmtp

Copilot uses AI. Check for mistakes.
Copy link
Author

Choose a reason for hiding this comment

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

@IEvangelist I don't have the technical capability to contribute to the aspire.dev repo. Is there a possibility to provide Markdown to be integrated by an existing contributor?

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