search

rio-labs / rio

WebApps in pure Python. No JavaScript, HTML and CSS needed
https://rio.dev
Apache License 2.0
453 stars 14 forks source link

Allow specifying additional files to watch #49

Closed mad-moo closed 2 weeks ago

mad-moo commented 3 weeks ago

rio run automatically restarts the project when changes to Python files are detected. Some projects also depend on other files however, such as JSON or CSV data files. It would be nice for rio run to watch those as well.

There's a couple approaches to solving this:

  1. Allow adding "negative" paths to `.rioignore.
  2. Add command-line flags for this
  3. Allow adding paths to rio.toml
mahiro72 commented 3 weeks ago

Hello @theVivern , I'm interested in working on this issue. Would it be okay for me to pick it up?

mad-moo commented 3 weeks ago

Hey @mahiro72 ! Really appreciate the offer. And yes, you absolutely can!

Let me know if you need help with anything.

mad-moo commented 2 weeks ago

The recent pull request has implemented this feature, but after some discussion (both internally & on discord) I've decided to rework the app files a bit. I'll use this as opportunity to drop .rioignore alltogether and improve rio.toml instead.

mad-moo commented 2 weeks ago

Fixed by 733d846d0181fdf40ea4cbfe58aedd1cd75adafe, and improved by subsequent commits