Skip to content

Share server submit option parsing#21

Closed
jolovicdev wants to merge 1 commit into
masterfrom
test/shared-submit-options-clean
Closed

Share server submit option parsing#21
jolovicdev wants to merge 1 commit into
masterfrom
test/shared-submit-options-clean

Conversation

@jolovicdev
Copy link
Copy Markdown
Owner

Summary

  • Move server submit option parsing into a shared helper used by sync and async handlers
  • Keep the existing submit defaults for cache, tags, retries, force, timeout, and ttl
  • Add server coverage for explicit cache: false

Testing

  • .venv/bin/ruff check src/ tests/
  • .venv/bin/pytest tests/ -q

Copy link
Copy Markdown

@ds-review ds-review Bot left a comment

Choose a reason for hiding this comment

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

DS-Review

No blocking issues found.

Reviewed the changed diff and related call paths. No actionable bugs, security issues, or meaningful performance regressions were found.

Copy link
Copy Markdown
Owner Author

Closing this as part of cleanup for temporary DS-Reviewer test branches. This PR was opened only for reviewer-bot scenario testing, not for merge.

@jolovicdev jolovicdev closed this May 10, 2026
@jolovicdev jolovicdev deleted the test/shared-submit-options-clean branch May 10, 2026 23:26
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.

1 participant