Tim Pillinger
Tim Pillinger
@oliver-sanders - are we waiting for a comment from @hjoliver ?
@oliver-sanders Please could you have a look at this proof of concept before I bother testing it?
> Much as I like the linter (I do!) we should think carefully, and maybe look at how widely it is used, before sinking a lot of effort into making...
Low priority - Branch & PR still available, but not currently worked on
> But I guess what Tim is describing is the workflow recording event times in local time, but job submission is still UTC? It's kind-of worse than that - the...
I think that I now have a test for the undelying cause: ```python @pytest.mark.parametrize( 'value,error', [ (None, TypeError), (42, TypeError) ] ) def test_get_unix_time_from_time_string_error(value, error): with pytest.raises(error): get_unix_time_from_time_string(value) def test_get_current_time_string():...
After discussion with @oliver-sanders I'm not _as_ sure that this is wrong as I was.
Posted to Element https://matrix.to/#/!gMuCeOJBhYqgWDGwnR:matrix.org/$174704625133144FCWJk:matrix.org?via=matrix.org
@oliver-sanders I may need some assistance working out why this change has caused very small changes in a couple of TUI tests:: 
> Also should remove any instances of time limit directives from the docs? E.g. > > https://github.com/cylc/cylc-flow/blob/86d24c3c1e63806590a89e75cc4202851feef1af/cylc/flow/job_runner_handlers/sge.py#L40 This appears to only be an issue for SGE where a not below...