Skip to content

Conversation

radical
Copy link
Member

@radical radical commented Apr 4, 2025

No description provided.

@radical radical requested a review from eerhardt as a code owner April 4, 2025 06:14
@github-actions github-actions bot added the area-engineering-systems infrastructure helix infra engineering repo stuff label Apr 4, 2025
Copy link
Contributor

@Copilot 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.

Copilot reviewed 2 out of 5 changed files in this pull request and generated no comments.

Files not reviewed (3)
  • tests/external-scripts/common.sh: Language not supported
  • tests/external-scripts/ubuntu-create-dotnet-devcert.sh: Language not supported
  • tests/helix/send-to-helix-inner.proj: Language not supported
Comments suppressed due to low confidence (1)

eng/pipelines/templates/BuildAndTest.yml:63

  • Ensure the file path matches the actual location of the script. The README is placed in tests/external-scripts, so if the script is also there, the pipeline reference should be updated accordingly.
      - script: $(Build.SourcesDirectory)/external-scripts/ubuntu-create-dotnet-devcert.sh

Copy link
Contributor

@Copilot 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.

Copilot reviewed 2 out of 5 changed files in this pull request and generated no comments.

Files not reviewed (3)
  • tests/external-scripts/common.sh: Language not supported
  • tests/external-scripts/ubuntu-create-dotnet-devcert.sh: Language not supported
  • tests/helix/send-to-helix-inner.proj: Language not supported
Comments suppressed due to low confidence (1)

eng/pipelines/templates/BuildAndTest.yml:63

  • Ensure that the local script 'ubuntu-create-dotnet-devcert.sh' is marked as executable in the repository (or explicitly set executable in the pipeline) to prevent CI failures.
script: $(Build.SourcesDirectory)/tests/external-scripts/ubuntu-create-dotnet-devcert.sh

Copy link
Member

@danmoseley danmoseley left a comment

Choose a reason for hiding this comment

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

thanks

@danmoseley danmoseley merged commit 85567dc into dotnet:main Apr 4, 2025
178 checks passed
@radical
Copy link
Member Author

radical commented Apr 5, 2025

/backport to release/9.2

Copy link
Contributor

github-actions bot commented Apr 5, 2025

Started backporting to release/9.2: https://github.com/dotnet/aspire/actions/runs/14278420973

@github-actions github-actions bot locked and limited conversation to collaborators May 5, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-engineering-systems infrastructure helix infra engineering repo stuff
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants