Skip to content

chore: remove migration-to-aws plugin#110

Closed
icarthick wants to merge 1 commit intoawslabs:mainfrom
icarthick:remove-migration-to-aws
Closed

chore: remove migration-to-aws plugin#110
icarthick wants to merge 1 commit intoawslabs:mainfrom
icarthick:remove-migration-to-aws

Conversation

@icarthick
Copy link
Copy Markdown
Contributor

Summary

  • Remove the migration-to-aws plugin (gcp-to-aws skill) and all associated files (31 files, ~3800 lines)
  • Remove all references from marketplace.json, AGENTS.md, README.md, and README.jp.md
  • CODEOWNERS entry intentionally retained

Test plan

  • mise run build passes (lint, format, manifests, cross-refs, reference integrity, size validation, security)
  • Grep confirms no remaining plugin references (except intentionally retained CODEOWNERS entry and an unrelated test query in databases-on-aws evals)

Generated with Claude Code


By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the project license.

Remove the migration-to-aws (gcp-to-aws) plugin and all references
from the marketplace, README, README.jp, and AGENTS.md.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@icarthick icarthick requested review from a team as code owners March 30, 2026 13:34
Copy link
Copy Markdown
Contributor

@theagenticguy theagenticguy left a comment

Choose a reason for hiding this comment

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

LGTM

@theagenticguy
Copy link
Copy Markdown
Contributor

@scottschreckengaust @krokoko can we also open the pr for claude-official-plugins please before we merge

@theagenticguy theagenticguy added the do-not-merge Do not merge the pull request label Mar 31, 2026
MichaelWalker-git added a commit that referenced this pull request Apr 1, 2026
Remove the migration-to-aws plugin (gcp-to-aws skill) and all associated
files. Clean up references from marketplace.json, AGENTS.md, README.md,
and README.jp.md. CODEOWNERS entry intentionally retained.

Supersedes #110 which had merge conflicts with main.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@rsmets
Copy link
Copy Markdown

rsmets commented Apr 1, 2026

@scottschreckengaust @krokoko can we also open the pr for claude-official-plugins please before we merge

Nice catch! I was going to call out the same thing... the ref inclaude-official-plugins needs to be removed.

@krokoko
Copy link
Copy Markdown
Contributor

krokoko commented Apr 1, 2026

Same for Cursor, did submit the request

@krokoko
Copy link
Copy Markdown
Contributor

krokoko commented Apr 15, 2026

Was removed from cursor

@krokoko
Copy link
Copy Markdown
Contributor

krokoko commented Apr 26, 2026

Closing this as duplicate of #114

@krokoko krokoko closed this Apr 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge Do not merge the pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants