Skip to content

exclude db from watch files extensions#5800

Merged
adhami3310 merged 1 commit into
mainfrom
exclude-db-from-watch-files-extensions
Sep 18, 2025
Merged

exclude db from watch files extensions#5800
adhami3310 merged 1 commit into
mainfrom
exclude-db-from-watch-files-extensions

Conversation

@adhami3310

Copy link
Copy Markdown
Member

No description provided.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Greptile Summary

This PR adds 'db' to the HOTRELOAD_IGNORE_EXTENSIONS tuple in reflex/utils/exec.py. This change prevents database files with the .db extension from triggering hot reloads during development in both Uvicorn and Granian backends.

The change integrates well with Reflex's existing hot reload system, which already ignores various file types that shouldn't cause application restarts (like .txt, .log, .json, .sqlite, etc.). Database files are frequently modified during normal application operation - whether through user interactions, background processes, or data synchronization - and these modifications don't represent code changes that require a server restart. By adding 'db' to the ignore list, the development experience becomes smoother as developers won't experience constant server restarts when their application's database is being actively used.

This follows the established pattern in Reflex where file extensions for data files, configuration files, and logs are excluded from hot reload monitoring. The change affects both the Uvicorn and Granian backend runners since they both use the same HOTRELOAD_IGNORE_EXTENSIONS constant.

Confidence score: 5/5

  • This PR is safe to merge with minimal risk
  • Score reflects a simple, well-justified addition that follows existing patterns and solves a common development friction point
  • No files require special attention

1 file reviewed, no comments

Edit Code Review Bot Settings | Greptile

@codspeed-hq

codspeed-hq Bot commented Sep 18, 2025

Copy link
Copy Markdown

CodSpeed Performance Report

Merging #5800 will not alter performance

Comparing exclude-db-from-watch-files-extensions (9b5cc9f) with main (8661f7e)

Summary

✅ 8 untouched

@adhami3310 adhami3310 merged commit 4468e14 into main Sep 18, 2025
41 checks passed
@adhami3310 adhami3310 deleted the exclude-db-from-watch-files-extensions branch September 18, 2025 21:55
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