Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Notebooks 2.0 // Backend // Enhance Local Development with Auto-Reload Functionality #135

Open
mohamedch7 opened this issue Nov 26, 2024 · 7 comments

Comments

@mohamedch7
Copy link

mohamedch7 commented Nov 26, 2024

Currently, we need to manually restart the server after making changes during development. This process can be time-consuming and interrupt the workflow. Implementing an autoreload tool (such as nodemon or an equivalent Go-based solution) will significantly improve the developer experience by automatically restarting the server when changes are detected.

Proposed Solution

Benefits

  • Improved developer productivity by removing the need for manual server restarts.
  • Streamlined development workflow, particularly for debugging and testing changes.

Demo

demo.mov
@ederign
Copy link
Member

ederign commented Dec 2, 2024

@mohamedch7 @thesuperzapper suggested that we investigate https://docs.tilt.dev/tutorial/index.html to see if it's a good fit for us.

Do you want to do this research and write a proposal?

@mohamedch7
Copy link
Author

@ederign yes

@ederign
Copy link
Member

ederign commented Dec 2, 2024

Perfect thank you! Feel free to repurpose this issue!

@KapilSareen
Copy link

Hi @ederign, I’d love to work on this issue! Could you clarify the decision between using tilt and nodemon?

Additionally, could you specify the type of restart required—should it watch for changes and restart locally (e.g., make run), or should it involve a rebuild and deploy to Docker? I’ll get started as soon as I have more details.

@KapilSareen
Copy link

/assign

@ederign
Copy link
Member

ederign commented Dec 16, 2024

@KapilSareen, please hold off on working on this issue until I define how to use tilt on the controller. I'm currently working on this. We are probably going to unify both approaches.

@KapilSareen KapilSareen removed their assignment Dec 18, 2024
@gaurangkudale
Copy link

Hi @ederign
As per your latest comment, Do you have any updates on this issue?
I'd like to work on this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Discussion
Development

No branches or pull requests

4 participants