Skip to content

Conversation

@ldionne
Copy link
Member

@ldionne ldionne commented Oct 27, 2025

I believe this was used at some point to automatically restart the development server when the LNT sources were changed. However, this configuration option doesn't appear to be hooked up anywhere, so this is effectively dead code.

Furthermore, Werzeug supports the use_reloader=True argument that could be used when we start a development server if we wanted this functionality (which we don't need at the moment).

I believe this was used at some point to automatically restart the
development server when the LNT sources were changed. However, this
configuration option doesn't appear to be hooked up anywhere, so this
is effectively dead code.

Furthermore, Werzeug supports the `use_reloader=True` argument that
could be used when we start a development server if we wanted this
functionality (which we don't need at the moment).
@ldionne ldionne merged commit 0b7112a into llvm:main Oct 28, 2025
15 checks passed
@ldionne ldionne deleted the review/remove-auto-restart branch October 28, 2025 21:06
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.

1 participant