Skip to content

Update Java resource dashboard icon to DrinkCoffee fluent icon#1223

Merged
maddymontaquila merged 1 commit intomainfrom
copilot/update-dashboard-icon-to-drinkcoffee
Mar 27, 2026
Merged

Update Java resource dashboard icon to DrinkCoffee fluent icon#1223
maddymontaquila merged 1 commit intomainfrom
copilot/update-dashboard-icon-to-drinkcoffee

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 20, 2026

Summary

Updates the Aspire dashboard icon for the Java resource and all its child resources to use the DrinkCoffee fluent icon (☕).

Changes

  • JavaAppHostingExtension.Container.cs: Added .WithIconName("DrinkCoffee") to both AddJavaContainerApp (primary method) and the obsolete AddJavaApp container overload.
  • JavaAppHostingExtension.Executable.cs: Added .WithIconName("DrinkCoffee") to:
    • AddJavaApp(name, workingDirectory) — the primary executable entry point
    • The obsolete AddJavaApp(name, workingDirectory, options) overload
    • WithJavaBuildStep — so Maven (MavenBuildResource) and Gradle (GradleBuildResource) child build resources also get the icon

Testing

  • Build succeeds with 0 errors and 0 warnings across net8.0 / net9.0 / net10.0 targets.

@github-actions
Copy link
Copy Markdown
Contributor

🚀 Dogfood this PR with:

⚠️ WARNING: Do not do this without first carefully reviewing the code of this PR to satisfy yourself it is safe.

curl -fsSL https://raw.githubusercontent.com/CommunityToolkit/Aspire/main/eng/scripts/dogfood-pr.sh | bash -s -- 1223

Or

  • Run remotely in PowerShell:
iex "& { $(irm https://raw.githubusercontent.com/CommunityToolkit/Aspire/main/eng/scripts/dogfood-pr.ps1) } 1223"

@aaronpowell
Copy link
Copy Markdown
Member

wait... when did I give @maddymontaquila permission to copilot this repo 😝

@maddymontaquila maddymontaquila merged commit 26f19a5 into main Mar 27, 2026
13 of 128 checks passed
@maddymontaquila maddymontaquila deleted the copilot/update-dashboard-icon-to-drinkcoffee branch March 27, 2026 23:23
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.

3 participants