Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 30, 2025

When pkgxdev installs curl for Codecov dependencies, it creates a .curlrc with unexpanded shell variables like ${SSL_CERT_FILE:-/github/home/.pkgx/curl.se/ca-certs/...}. This mirrors the existing gpgconf.ctl issue.

Changes

  • Created new actions/codecov/action.yml - dedicated action for Codecov integration
  • Extracted all Codecov-related logic from actions/test/action.yml to the new action
  • Added "Fix curl configuration" step that removes malformed .curlrc when curl is installed via pkgxdev
  • Added "Fix GPG configuration" step that removes malformed gpgconf.ctl
  • Added cleanup step to uninstall pkgxdev dependencies after Codecov upload completes
  • Added actions/codecov/README.md documentation
  • Updated main README.md to include the new codecov action

New Codecov Action Features

  • Automatic dependency management: Detects and installs missing dependencies (Git, cURL, gnupg) using pkgxdev regardless of context
  • Configuration fixes: Automatically fixes pkgxdev configuration issues with unexpanded environment variables
  • Cleanup: Uninstalls dependencies after Codecov upload is complete
  • OIDC support: Uses OIDC authentication with Codecov for secure uploads
Original prompt

On action.yml:175-175, This variable is not expended as well, fix it:


💡 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 AI self-assigned this Nov 30, 2025
Copilot AI changed the title [WIP] Fix variable expansion issue in action.yml fix(test): handle unexpanded SSL_CERT_FILE in pkgxdev curl config Nov 30, 2025
Copilot AI requested a review from neilime November 30, 2025 08:56
@github-actions
Copy link
Contributor

github-actions bot commented Nov 30, 2025

Code Coverage Report

Coverage Results

Coverage

Metric Covered Total Percentage
Lines 2 2 100.00%
Functions 1 1 100.00%

Overall: 100.00% 🟢
🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩

Copy link
Member

@neilime neilime left a comment

Choose a reason for hiding this comment

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

@copilot handle feedbacks

Copy link
Contributor Author

Copilot AI commented Nov 30, 2025

@copilot handle feedbacks

Done. Created actions/codecov/action.yml with all Codecov logic and added dependency cleanup step. See commit 2d0587f.

Copilot AI changed the title fix(test): handle unexpanded SSL_CERT_FILE in pkgxdev curl config refactor(test): extract Codecov logic to dedicated action with config fixes Nov 30, 2025
Copilot AI requested a review from neilime November 30, 2025 09:43
Copy link
Member

@neilime neilime left a comment

Choose a reason for hiding this comment

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

@copilot handle feedbacks

Copy link
Contributor Author

Copilot AI commented Nov 30, 2025

@copilot handle feedbacks

Done. Removed container input and dependency check now runs unconditionally. Commit 9f33458.

Copilot AI requested a review from neilime November 30, 2025 09:53
@neilime neilime marked this pull request as ready for review November 30, 2025 11:15
@neilime neilime force-pushed the copilot/fix-variable-expansion-issue branch from 9f33458 to d245700 Compare November 30, 2025 11:21
@neilime neilime merged commit 4e4a77f into main Nov 30, 2025
66 checks passed
@neilime neilime deleted the copilot/fix-variable-expansion-issue branch November 30, 2025 12:37
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