Simon de Vlieger

Results 101 comments of Simon de Vlieger

Yea that helps, the `--explain` functionality I hadn't seen yet and that pretty much negates the first suggestion. For the second one I'm on track to get you a PR,...

I'll give it a try, I've got a bunch of them lying around and a programmer. Thanks for the pointers!

@bcl @gicmo Would we prefer all these checks (so also the mypy checks) to be rolled into the container + test-src?

I'll take that and merge the current outstanding linting-related PRs into the CI container and a single PR here.

@bcl, @gicmo this PR has been updated to use the `test-src` Makefile target instead but will now be dependent on this container PR to be merged: https://github.com/osbuild/containers/pull/36

This needs a final review (just a rebase with a conflict on the version number) to merge.

Moved this to `utils/`, added `__getattr__` to naively disable the formatting when stdout is not a tty. I know that doesn't catch all cases but it also doesn't make things...

Sounds good, I'll get the changes in soon :)

Merci for the review. - Renamed to `term.py`. - Instance name is now `fmt` from `term` (I don't like duplicating VT/term). - `isatty` is now cached on the `VT` instance.