Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 26, 2025

  • Change (factoryContext) to _ in PipelineStepAnnotation lambda (unused parameter)
  • Change resource.Name to _targetResource.Name (use field instead of parameter)
  • Change composeEnvironmentResource to _composeEnvironmentResource (use field instead of parameter)
  • Remove unnecessary dockerComposeFilePath declaration and file existence check in PrintEndpointsAsync

💡 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.

@captainsafia captainsafia marked this pull request as ready for review November 26, 2025 21:49
@captainsafia captainsafia self-requested a review as a code owner November 26, 2025 21:50
@captainsafia captainsafia merged commit 40a62e5 into safia/docker-compose-endpoints Nov 26, 2025
2 of 3 checks passed
@captainsafia captainsafia deleted the copilot/sub-pr-13216 branch November 26, 2025 21:50
davidfowl pushed a commit that referenced this pull request Dec 7, 2025
* Display resource endpoints for Docker Compose deploy

* [WIP] Update resource endpoints display for Docker Compose deploy (#13217)

* Initial plan

* Address review feedback on DockerComposeServiceResource

Co-authored-by: captainsafia <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: captainsafia <[email protected]>

---------

Co-authored-by: Copilot <[email protected]>
Co-authored-by: captainsafia <[email protected]>
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