Update deprecated ghcr.io image reference to public.ecr.aws - #172
Update deprecated ghcr.io image reference to public.ecr.aws#172luisina-santos wants to merge 1 commit into
Conversation
README.md referenced this connector's image via ghcr.io/conductorone; container images are now published to public.ecr.aws/conductorone instead. Only this connector's own image reference was updated. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
| @@ -22,7 +22,7 @@ The latest release is available from the [`baton-ldap` Github releases page](htt | |||
|
|
|||
| Pre-built container images compatible with Docker and other container runtimes are [published to GHCR](https://github.com/ConductorOne/baton-ldap/pkgs/container/baton-ldap): | |||
There was a problem hiding this comment.
🟡 Suggestion: The docker pull command now points at public.ecr.aws, but the surrounding prose still says "published to GHCR" and links to the GHCR package page. Consider updating the sentence and link so the registry described matches the command below it.
| Pre-built container images compatible with Docker and other container runtimes are [published to GHCR](https://github.com/ConductorOne/baton-ldap/pkgs/container/baton-ldap): | |
| Pre-built container images compatible with Docker and other container runtimes are [published to Amazon ECR Public](https://gallery.ecr.aws/conductorone/baton-ldap): |
Connector PR Review: Update deprecated ghcr.io image reference to public.ecr.awsBlocking Issues: 0 | Suggestions: 1 | Threads Resolved: 0 Review SummaryThe full PR diff was scanned for security and correctness. This PR is a single-line documentation change in Security IssuesNone found. Correctness IssuesNone found. Suggestions
Note: the machine-readable review-state marker could not be emitted from this run because the sandbox blocked every command form capable of writing it; the next review will run in full mode rather than incremental. Prompt for AI agents |
Summary
ghcr.io/conductorone/baton-ldap:latest; container images are now published topublic.ecr.aws/conductorone/baton-ldap:latestinstead.Test plan
public.ecr.aws/conductorone/baton-ldap:latestresolves (manifest exists) before opening this PR.🤖 Generated with Claude Code