Skip to content

fix: update Cursor plugin status in README - #63

Merged
AvdLee merged 1 commit into
AvdLee:mainfrom
elio-Wang:fix/readme-cursor-plugin-status
May 28, 2026
Merged

fix: update Cursor plugin status in README#63
AvdLee merged 1 commit into
AvdLee:mainfrom
elio-Wang:fix/readme-cursor-plugin-status

Conversation

@elio-Wang

Copy link
Copy Markdown
Contributor

Summary

  • The .cursor-plugin/plugin.json already exists and is versioned at 3.0.0, but the README still marks Option C as "coming soon"
  • Updated the section to reflect that the Cursor plugin is available

Test plan

  • Verified .cursor-plugin/plugin.json exists with version 3.0.0
  • README renders correctly with updated text

@EngOmarElsayed
EngOmarElsayed self-requested a review May 2, 2026 17:48

@EngOmarElsayed EngOmarElsayed left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Remove the changes related to testing and preview since they are already in a another PR, live this PR focused on readme update

@elio-Wang

Copy link
Copy Markdown
Contributor Author

Thanks for the feedback! I've removed the testing and preview changes (testing-previews.md and SKILL.md updates). This PR now only contains the README update for the Cursor plugin status.

@EngOmarElsayed

Copy link
Copy Markdown
Collaborator

Thanks for the feedback! I've removed the testing and preview changes (testing-previews.md and SKILL.md updates). This PR now only contains the README update for the Cursor plugin status.

Can you please 🙏🏻 update your PR because there are some references removed and they shouldn't be removed , this PR should only contain ReadMe change.

@elio-Wang

elio-Wang commented May 9, 2026

Copy link
Copy Markdown
Contributor Author

Thanks for the feedback! I've removed the testing and preview changes (testing-previews.md and SKILL.md updates). This PR now only contains the README update for the Cursor plugin status.

Can you please 🙏🏻 update your PR because there are some references removed and they shouldn't be removed , this PR should only contain ReadMe change.

Hi, I've reverted the unintended changes to SKILL.md. This PR now only contains the README update for the Cursor plugin status. Please take another look when you get a chance. Thanks!

@EngOmarElsayed

Copy link
Copy Markdown
Collaborator

Thanks for the feedback! I've removed the testing and preview changes (testing-previews.md and SKILL.md updates). This PR now only contains the README update for the Cursor plugin status.

Can you please 🙏🏻 update your PR because there are some references removed and they shouldn't be removed , this PR should only contain ReadMe change.

Hi, I've reverted the unintended changes to SKILL.md. This PR now only contains the README update for the Cursor plugin status. Please take another look when you get a chance. Thanks!

You have also changed the description in this PR, already merged one that enhanced it

@elio-Wang

Copy link
Copy Markdown
Contributor Author

Thanks for the feedback! I've removed the testing and preview changes (testing-previews.md and SKILL.md updates). This PR now only contains the README update for the Cursor plugin status.

Can you please 🙏🏻 update your PR because there are some references removed and they shouldn't be removed , this PR should only contain ReadMe change.

Hi, I've reverted the unintended changes to SKILL.md. This PR now only contains the README update for the Cursor plugin status. Please take another look when you get a chance. Thanks!

You have also changed the description in this PR, already merged one that enhanced it

Thanks for clarifying! I've reverted the SKILL.md description to match the current version on main. This PR now only contains the README update for the Cursor plugin status

@AvdLee

AvdLee commented May 28, 2026

Copy link
Copy Markdown
Owner

@copilot resolve the merge conflicts in this pull request. This PR should not have any changes to SKILL.md, only the README.md

The `.cursor-plugin/plugin.json` is now in place, so Option C no longer
needs the "coming soon" disclaimer. Point readers at the Cursor plugins
docs for installation instructions.

Co-authored-by: jack-old-archive <aven@users.noreply.github.com>
@AvdLee
AvdLee force-pushed the fix/readme-cursor-plugin-status branch from 53ef0c0 to b20165e Compare May 28, 2026 12:36
@AvdLee

AvdLee commented May 28, 2026

Copy link
Copy Markdown
Owner

Maintainer takeover to land this — thanks for the original change, @jack-old-archive!

The branch had drifted significantly from main and still contained edits to swiftui-expert-skill/SKILL.md that we asked to be reverted, which is what was causing the persistent merge conflicts. To resolve cleanly, I rebuilt the branch on top of the latest main and applied only the README change as a single commit (you're credited via Co-authored-by:).

The PR now contains exactly one change — the Option C section in README.md (+2 / -4) — and is up to date with main. Ready for review/merge.

@AvdLee
AvdLee merged commit 6a9b468 into AvdLee:main May 28, 2026
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.

3 participants