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

Triggers modified_files on file deletion #2147

Merged

Conversation

hernan604
Copy link

Summary

Updates Mojo::Server::Morbo::Backend::Poll to detect deleted files.

Motivation

Currently Mojo::Server::Morbo::Backend::Poll does not restart when files are deleted.

References

No references currently

@hernan604 hernan604 force-pushed the restart_morbo_when_file_is_deleted branch 2 times, most recently from 2a63944 to 10cde32 Compare February 4, 2024 19:39
@kraih
Copy link
Member

kraih commented Feb 4, 2024

This PR is missing a test.

@hernan604
Copy link
Author

This PR is missing a test.

Added a test

@hernan604 hernan604 force-pushed the restart_morbo_when_file_is_deleted branch 2 times, most recently from e0e2c50 to f537737 Compare February 5, 2024 00:16
t/mojo/morbo.t Outdated Show resolved Hide resolved
@hernan604 hernan604 force-pushed the restart_morbo_when_file_is_deleted branch 3 times, most recently from 1fdfb69 to d3063be Compare February 5, 2024 23:46
@kraih
Copy link
Member

kraih commented Feb 6, 2024

I'm in favour once all issues are resolved.

@jberger
Copy link
Member

jberger commented Feb 6, 2024

I'm in favour once all issues are resolved.

Seconded, fix the issues and I'm in too.

@hernan604 hernan604 force-pushed the restart_morbo_when_file_is_deleted branch from d3063be to 632ae27 Compare February 6, 2024 22:28
@hernan604
Copy link
Author

I think all issues have been resolved. If i missed something, let me know. Thanks

Copy link
Member

@marcusramberg marcusramberg left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Member

@jhthorsen jhthorsen left a comment

Choose a reason for hiding this comment

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

Looks good to me as well 👍

@kraih kraih merged commit 37a4c39 into mojolicious:main Feb 9, 2024
9 of 10 checks passed
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.

5 participants