Marek Kaput
Marek Kaput
`cairo-format` is not a linter but code formatter. I suggest renaming the feature to `protostar fmt` and let it behave the same as `cargo fmt` or `mix format`. Wdyt? @kasperski95...
Also, I think that the write mode (`-i` switch) should be on by default, with the `check` mode (`-c`) being an option. I don't see an use case for a...
Yeah, `protostar format` is better here.
> I'd go with the safe by default approach. You can always change the behavior in the `protostar.toml`. By default, all flags in Protostar are off/false, so we should consider...
> Checking if the project is correctly formatted (CI check). `protostar format --fix` could overwrite files. > > Anyway, we should do a research and make it work similarly to...
> > IDEs should call cairo-format directly if they want to format a file > > Users would have to set up the environment. Maybe we should create a Protostar...
@machard Not only to minimize merge conflicts, we would just like to be aligned with the ongoing work in test runner 😄 Its internals are not documented anywhere, and we...
> * Protostar probably parse cairo file to AST ? In this case I was thinking to instrument on the fly here Yes, we do even modify the Cairo compiler...
Hi @ametel01 did @kasperski95's suggestion fix the issue for you? Closing this issue as for now.