Skip to content

Replace peter-evans/dockerhub-description with direct Docker Hub API call#2966

Draft
Copilot wants to merge 1 commit intomasterfrom
copilot/analyze-replace-dockerhub-actions
Draft

Replace peter-evans/dockerhub-description with direct Docker Hub API call#2966
Copilot wants to merge 1 commit intomasterfrom
copilot/analyze-replace-dockerhub-actions

Conversation

Copy link
Copy Markdown

Copilot AI commented Apr 23, 2026

Removes the third-party peter-evans/dockerhub-description action from the push-container-images workflow, replacing it with a native PowerShell call to the Docker Hub REST API. snok/container-retention-policy is not used in this repo.

Changes

  • push-container-images.yml: Replace peter-evans/dockerhub-description@v5.0.0 with a run step that:
    • Authenticates via POST https://hub.docker.com/v2/users/login/ using existing DOCKERHUB_USERNAME/DOCKERHUB_TOKEN secrets
    • Masks the returned JWT with ::add-mask:: to prevent log exposure
    • Reads Container-README.md and updates the repository via PATCH https://hub.docker.com/v2/repositories/particular/servicepulse/
    • Both Invoke-RestMethod calls use -ErrorAction Stop for explicit failure reporting

Reviewer Checklist

  • Components are broken down into sensible and maintainable sub-components.
  • Styles are scoped to the component using it. If multiple components need to share CSS, then a .css file is created containing the shared CSS and imported into component scoped style sections.
  • Naming is consistent with existing code, and adequately describes the component or function being introduced
  • Only functions utilizing Vue state or lifecycle hooks are named as composables (i.e. starting with 'use');
  • No module-level state is being introduced. If so, request the PR author to move the state to the corresponding Pinia store.

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