Skip to content

no sync after install for reflex-web#5431

Merged
adhami3310 merged 3 commits into
mainfrom
no-sync-after-install-for-reflex-web
Jun 11, 2025
Merged

no sync after install for reflex-web#5431
adhami3310 merged 3 commits into
mainfrom
no-sync-after-install-for-reflex-web

Conversation

@adhami3310
Copy link
Copy Markdown
Member

No description provided.

Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

PR Summary

Modified integration test workflow to add --no-sync flag in the reflex-web ubuntu test job, preventing package synchronization during installation and init phases.

  • Added --no-sync flag to uv run commands in .github/workflows/integration_tests.yml for ubuntu reflex-web tests
  • Created platform-specific behavior discrepancy between ubuntu and macOS test workflows
  • Potential impact on test consistency across different operating systems

1 file reviewed, 2 comments
Edit PR Review Bot Settings | Greptile

Comment thread .github/workflows/integration_tests.yml Outdated
Comment thread .github/workflows/integration_tests.yml Outdated
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Jun 11, 2025

CodSpeed Performance Report

Merging #5431 will not alter performance

Comparing no-sync-after-install-for-reflex-web (a35e8e8) with main (e9cd7e0)

Summary

✅ 8 untouched benchmarks

Lendemor
Lendemor previously approved these changes Jun 11, 2025
@adhami3310 adhami3310 merged commit bc6ad72 into main Jun 11, 2025
41 checks passed
@adhami3310 adhami3310 deleted the no-sync-after-install-for-reflex-web branch June 11, 2025 01:46
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