[Proposal] Add flag to `mix format` to specify timeout

41 views
Skip to first unread message

Marc Noordman

unread,
Mar 2, 2023, 9:52:05 AM3/2/23
to elixir-lang-core
We have a large Elixir project that runs `mix format --check-formatted` in the CI pipeline.
Unfortunately this command sometimes times out, due to project size and server load. 
This makes our CI pipeline result unreliable.

It turns out that there is a hard-coded 30 second timeout for mix format specified in 
on line 228. 
My proposal would be to make the timeout duration customizable with an optional flag to `mix format`, so we can do something like
`mix format --check-formatted --timeout 300`
to give mix format 5 minutes instead of 30 seconds to check the code. The default, when the flag not provided, should still be to time out after 30 seconds. 

José Valim

unread,
Mar 2, 2023, 1:19:45 PM3/2/23
to elixir-l...@googlegroups.com
Timeout bumped to infinity on both main and v1.14.

--
You received this message because you are subscribed to the Google Groups "elixir-lang-core" group.
To unsubscribe from this group and stop receiving emails from it, send an email to elixir-lang-co...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/elixir-lang-core/c91e03ea-49f6-4438-9114-4e7193d262f4n%40googlegroups.com.
Reply all
Reply to author
Forward
0 new messages