Dorian Péron comments

Results 32 comments of


                                            Dorian Péron

ls: fix quoting alignment, add tests for default quoting style in TTY and quoting alignment

> I guess you saw that some jobs are failing Yes, sorry for that. I always forget about linting and windows stuff.

ls: fix quoting alignment, add tests for default quoting style in TTY and quoting alignment

Forgot to add `#[cfg(unix)]` on all TTY tests.

ls: fix quoting alignment, add tests for default quoting style in TTY and quoting alignment

@sylvestre if you don't mind taking look. I don't understand why is the macos test failing.

date: format date to correct date format before parsing

After looking at the code, there is [this snippet](https://github.com/uutils/parse_datetime/blob/138999131b302f12775d4baf9802b85b9727fab9/src/lib.rs#L207-L216) in the `parse_datetime_at_date` function that checks `1999-1-4` against the ISO 8601 pattern. For some reasons (I may not have all the...

date: format date to correct date format before parsing

Yes, it most probably is.

date: format date to correct date format before parsing

The following example isolates the bug being on the parsing of the last single digit. It has no issue parsing `-1-` as a month. ```rust use parse_datetime::parse_datetime; fn main() {...

date: format date to correct date format before parsing

Here is a PR for the fix in `parse_datetime` https://github.com/uutils/parse_datetime/pull/76

date: format date to correct date format before parsing

Waiting for https://github.com/uutils/parse_datetime/pull/76 to be released

test utilities: easy way to simulate terminal context

Is it ready to merge already, or are we waiting for further improvements ?

Failing to load theme

It looks like the version of json-editor used is the 1.3.5, which is 5 years old. This is the most probable reason as of why isn't this theme available.