Rename spock.seq_sync to spock.sync_seq#359
Rename spock.seq_sync to spock.sync_seq#359zaidshabbir25 wants to merge 2 commits intopgEdge:mainfrom
Conversation
I verified this with Spock 5.0.5 and could not find any function named seq_sync. However, a similar function named sync_seq is present.
📝 WalkthroughWalkthroughDocumentation for the spock sequence synchronization function was updated to rename the public API from Changes
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/spock_functions/functions/spock_seq_sync.md`:
- Line 1: Rename the documentation file
docs/spock_functions/functions/spock_seq_sync.md to
docs/spock_functions/functions/spock_sync_seq.md so the filename matches the
documented function spock.sync_seq(), and update all references to the function
in the site config and generated docs—specifically replace any occurrences of
spock.seq_sync() with spock.sync_seq() in mkdocs.yml and in
docs/spock_functions/gen_mgmt.md (and any other references) so the navigation
and links point to the correct function name.
I verified this with Spock 5.0.5 and could not find any function named seq_sync. However, a similar function named sync_seq is present.