Is there any easy

fix

command to run linters across the entire project and have them update files in place?

this doesn’t currently exist, but this is a great idea! what kinds of fixes would you have in mind?

Mostly the

goimports

and

gci

since I'm working on a backend plugin. I try to run the linter from

make

locally but it errors out when it notices I already have some binaries downloaded. Looks like the scripts are designed to be run from a fresh install on CI every time. There may be some room for improvement there too i guess

does

make goimports

work?

Yeah.

make goimports

does work, although it results in very noisy changes (flytectl, generated files, etc). Ah I see that comes from an imported

Makefile

nice

if it’s coming from the boilerplate repo, feel free to override (just add a comment)

@thankful-minister-83577, a minor correction: boilerplate was moved to the monorepo (in fact the original repo is marked as readonly), so making this change is just a matter of fixing

goimports

in the flyte repo.

I think I have an elegant solution here? https://github.com/flyteorg/flyte/pull/5484

@clean-glass-36808, thanks for your PR. Left one small comment.

Thanks for the feedback, just pushed an update