Skip to content

Conversation

@jeromelaban
Copy link
Member

No description provided.

Copilot AI review requested due to automatic review settings December 2, 2025 20:02
@jeromelaban jeromelaban merged commit b42114a into main Dec 2, 2025
3 of 5 checks passed
Copy link

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 refactors the NuGet package publishing process in the GitHub Actions workflow by replacing the previous two-step approach (adding a source, then pushing) with a streamlined single-step push command using environment variables for configuration.

Key Changes:

  • Replaced explicit source addition with environment variable configuration
  • Consolidated two dotnet nuget commands into a single push command
  • Added --skip-duplicate flag to handle existing packages gracefully

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

run: |
dotnet nuget add source https://pkgs.dev.azure.com/uno-platform/1dd81cbd-cb35-41de-a570-b0df3571a196/_packaging/unoplatformdev/nuget/v3/index.json -n "dev" --username az --password "${{ inputs.token }}" --store-password-in-clear-text
dotnet nuget push --api-key AZ -s dev artifacts/*.nupkg
dotnet nuget push artifacts/*.nupkg --source $env:FEED_ENDPOINT --api-key az --skip-duplicate
Copy link

Copilot AI Dec 2, 2025

Choose a reason for hiding this comment

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

The API key 'az' appears to be a placeholder or incorrect value. Azure DevOps feeds typically require the actual token value as the API key. Consider using ${{ inputs.token }} as the API key value instead of 'az'.

Suggested change
dotnet nuget push artifacts/*.nupkg --source $env:FEED_ENDPOINT --api-key az --skip-duplicate
dotnet nuget push artifacts/*.nupkg --source $env:FEED_ENDPOINT --api-key "${{ inputs.token }}" --skip-duplicate

Copilot uses AI. Check for mistakes.
Copilot finished reviewing on behalf of jeromelaban December 3, 2025 15:50
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