Skip to content

Conversation

kieran-osgood-shopify
Copy link
Contributor

@kieran-osgood-shopify kieran-osgood-shopify commented Jul 15, 2025

What changes are you making?

Adding some cursor rules that have been helpful in getting cursor to do less bad swift

How to test


Before you merge

Important


Checklist for releasing a new version
  • I have bumped the version number in the podspec file.
  • I have bumped the version number in the README.
  • I have added a Changelog entry.

Tip

See the Contributing documentation for instructions on how to publish a new version of the library.

@kieran-osgood-shopify kieran-osgood-shopify requested a review from a team as a code owner July 15, 2025 10:46
Copy link
Contributor

@tiagocandido tiagocandido left a comment

Choose a reason for hiding this comment

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

+1 on Joey suggestions, but it lgtm

@kieran-osgood-shopify kieran-osgood-shopify force-pushed the kieran-osgood/chore/add-cursor-rules branch from ca09ea4 to 22a65bc Compare July 24, 2025 10:22
@kieran-osgood-shopify kieran-osgood-shopify force-pushed the kieran-osgood/chore/add-cursor-rules branch from 22a65bc to 37b64d2 Compare July 24, 2025 10:22
@kieran-osgood-shopify kieran-osgood-shopify merged commit 2db41d0 into main Jul 24, 2025
7 checks passed
@kieran-osgood-shopify kieran-osgood-shopify deleted the kieran-osgood/chore/add-cursor-rules branch July 24, 2025 11:28
tiagocandido pushed a commit that referenced this pull request Jul 25, 2025
* feat: add cursor rules for improving usage on swift

* Apply suggestions from code review

Co-authored-by: Mark Murray <[email protected]>

* fix: update mdc files to add more context and refactor wording

* refactor: add more examples for cursor rules and move dev descriptions to dev.yml

---------

Co-authored-by: Mark Murray <[email protected]>
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.

4 participants