Skip to content

INTER-1793: Improve sync script#157

Merged
erayaydin merged 1 commit intomainfrom
inter-1793-improve-sync
Feb 5, 2026
Merged

INTER-1793: Improve sync script#157
erayaydin merged 1 commit intomainfrom
inter-1793-improve-sync

Conversation

@erayaydin
Copy link
Member

Summary

Improve the sync.sh script with better error handling, retry logic, and reduced code duplication.

Changes

  • Add set -euo pipefail for strict error handling and early-fail
  • Add --retry 3 to curl commands for network resilience
  • Use curl -fSL flags for better error handling and redirect following
  • Add configurable URLs via script arguments, with the current one as the default
  • Add mkdir -p to ensure directories exist before writing
  • Add download_example helper function to reduce duplication
  • Use proper bash array iteration

Add strict error handling, retry logic, and reduce code duplication.

Related-Task: INTER-1793
@erayaydin erayaydin self-assigned this Feb 3, 2026
@erayaydin erayaydin added the enhancement New feature or request label Feb 3, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 3, 2026

⚠️ This PR doesn't contain any changesets. If there are user-facing changes, don't forget to run:

pnpm exec changeset

to create a changeset.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 3, 2026

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
5857 4927 84% 0% 🟢

New Files

No new covered files...

Modified Files

No covered modified files...

updated for commit: c97726d by action🐍

@erayaydin erayaydin marked this pull request as ready for review February 3, 2026 18:46
@erayaydin erayaydin merged commit b3c1f80 into main Feb 5, 2026
24 checks passed
@erayaydin erayaydin deleted the inter-1793-improve-sync branch February 5, 2026 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments