Skip to content

fix: update Cursor plugin status in README#63

Open
Jack-sh1 wants to merge 6 commits into
AvdLee:mainfrom
Jack-sh1:fix/readme-cursor-plugin-status
Open

fix: update Cursor plugin status in README#63
Jack-sh1 wants to merge 6 commits into
AvdLee:mainfrom
Jack-sh1:fix/readme-cursor-plugin-status

Conversation

@Jack-sh1
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

…terns

The skill currently has no coverage for testing or previews. Add a new
reference covering #Preview macro usage and traits, @Previewable for
inline bindings, mock data patterns for previews, XCTest strategies
for @observable models and UI tests, ViewInspector for view-layer
unit testing, and snapshot testing with swift-snapshot-testing.
The Cursor plugin configuration (.cursor-plugin/plugin.json) already
exists and is versioned at 3.0.0, but the README still marked it as
'coming soon'. Update the section to reflect the current state.
@EngOmarElsayed EngOmarElsayed self-requested a review May 2, 2026 17:48
Copy link
Copy Markdown
Collaborator

@EngOmarElsayed EngOmarElsayed left a comment

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

@Jack-sh1
Copy link
Copy Markdown
Contributor Author

Jack-sh1 commented May 2, 2026

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.

@Jack-sh1
Copy link
Copy Markdown
Contributor Author

Jack-sh1 commented May 9, 2026

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

@Jack-sh1
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

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