Miloš Prchlík
Miloš Prchlík
I believe this was delivered already, in tmt 1.49.
> Pretty sure @happz would very much not like this as ruff formatter is made to be drop-in replacement for black, not autopep8. Indeed, I for one was never fond...
Added to our meeting agenda, let's get some input.
> One more thing: Among the changes I see also these: > > ```diff > dynamic_ref = resolve_dynamic_ref( > - workdir=destination, > - ref=plan_id.ref, > - plan=self, > - logger=self._logger...
> > @psss what do you think about this limitation? I.e. > > > there's no option to change that behavior because we intentionally limit the formatter options and only...
> I've quickly reviewed changes performed by `ruff-format` and overall I'd say they are reasonable. There are some cases though, where the original multi-line formatting is compressed into a single...
> It's an additional work, yes, and it's not just a few. On the other hand it's just a one time task. When writing new code we can keep this...
> > I'm clearly in opposition to the proposal > > Wait, @happz is against ruff format in general? No, just against keeping humans responsible for formatting by using trailing...
> If users specify memory: > 4096 ,when they want to get an instance with memory > 4096 MiB Maybe, who knows what user wanted :) The should have been...
> If users specify memory: > 4096 ,when they want to get an instance with memory > 4096 MiB Maybe, who knows what user wanted :) The should have been...