Skip to content

Conversation

Copilot
Copy link
Contributor

@Copilot Copilot AI commented Aug 12, 2025

This PR normalizes the naming of "app host" to "AppHost" across all .NET Aspire project templates to ensure consistency with the documentation and improve the developer experience.

Changes Made

Updated template descriptions in the following templates to use "AppHost" instead of "app host":

  • aspire-apphost: Changed description from "A project template for creating a .NET Aspire app host (orchestrator) project." to "A project template for creating a .NET Aspire AppHost (orchestrator) project."
  • aspire-mstest: Changed description from "A project that contains MSTest integration tests of a .NET Aspire app host project." to "A project that contains MSTest integration tests of a .NET Aspire AppHost project."
  • aspire-nunit: Changed description from "A project that contains NUnit integration tests of a .NET Aspire app host project." to "A project that contains NUnit integration tests of a .NET Aspire AppHost project."

Files Updated

  • Template configuration files (template.json) for aspire-apphost, aspire-mstest, and aspire-nunit
  • English localized strings (templatestrings.en.json) for all affected templates
  • Italian localized strings (templatestrings.it.json) for aspire-nunit template
  • Fixed BOM encoding issue in the Italian localization file

Impact

This change ensures consistent naming across all .NET Aspire templates where "AppHost" is always capitalized as one word, aligning with the documentation standards and providing a better developer experience. The aspire-xunit template already used the correct "AppHost" naming and did not require changes.

Fixes #10887.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@Copilot Copilot AI changed the title [WIP] Normalize "AppHost" for templates Normalize "AppHost" naming in project templates Aug 12, 2025
@Copilot Copilot AI requested a review from maddymontaquila August 12, 2025 17:00
Copilot finished work on behalf of maddymontaquila August 12, 2025 17:00
@maddymontaquila
Copy link
Member

@DamianEdwards is this how the localization works??? seems like copilot edited some strings.en.json but not others?

@DamianEdwards
Copy link
Member

Yes. There's a separate process for updating translations that's automated I believe.

@maddymontaquila
Copy link
Member

@copilot please undo your changes to locale-specific files, changes should only be in template.json

@maddymontaquila
Copy link
Member

ill just do this one myself because copilot is being annoying

@maddymontaquila maddymontaquila deleted the copilot/fix-10887 branch August 21, 2025 18:26
@dotnet-policy-service dotnet-policy-service bot added this to the 9.5 milestone Aug 21, 2025
@maddymontaquila maddymontaquila removed this from the 9.5 milestone Aug 21, 2025
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.

Normalize "AppHost" for templates
3 participants