Hello, Has anyone encountered discrepancy in output when running

flake8

on its own and comparing it with

./pants lint ::

? For example, when I run

flake8

, I get:

file1.py:17:21: ANN201 Missing return type annotation for public function
file2.py:10:32: ANN201 Missing return type annotation for public function

However, when I run

./pants lint ::

, I get:

11:38:23.22 [INFO] Completed: Lint with Flake8 - flake8 succeeded.
...
✓ flake8 succeeded.

Is there a way to get more verbose output of

./pants lint ::

? So as to see what files are linted, what flake8 config, etc.

You can run Pants in debug log mode:

-ldebug

Seems like

flake8

in Pants is either: • Missing the plugin(s) which report that code. Have you added the plugin to https://www.pantsbuild.org/docs/reference-flake8#section-extra-requirements? • Missing the relevant config file(s) which enable the code

Yeah, that's a good hunch Josh. That error code is not coming from core flake8

@bitter-ability-32190 it’s most likely 1, since we use a bunch of flake8 plugins. Those plugins are defined in pyproject.toml for poetry. Any way to bridge the two without specifying plugins in two places (i.e., pants.toml and pyproject.toml)

(FWIW I find it surprising the default lockfile is provided for you. And worse, it has more than just` flake8` in it 😐)

it shouldn't @bitter-ability-32190. only this

default_version = "flake8>=3.9.2,<4.0"

Pants uses its own internal flake8 lockfile

@full-student-91825 it depends on your setup. If you hand the keys solely over to Pants, then you only need to define them in Pants' ecosystem. If you're trying to straddle both Pants and Poetry I think you're SoL. Others might know more about straddling the two though

Any way to bridge the two without specifying plugins in two places (i.e., pants.toml and pyproject.toml)

Generally, you can use the poetry_requirements target generator to bridge the gap. Unfortunately, that is not yet integrated with tools https://github.com/pantsbuild/pants/issues/12449

@hundreds-father-404 re: lockfile: oh that's nuanced. I dind't realize that

Which comment are you replying to?

(but now I understand the "synthetic lockfile". Also sorry Batyr for side convo)

@bitter-ability-32190 np, i am learning too 😄

fyi I commented onhttps://github.com/pantsbuild/pants/issues/12449 with your use case No pressure at all, but if you are interested, it is marked "good first issue". It would allow you to solve this problem without duplicating the requirement strings. We would be super happy to help you implement it, this is a very requested feature

@hundreds-father-404 sounds good, you can assign it to me (gh handle: oneturkmen) and I can take a look at it this weekend 🙂 (can’t comment on the issue atm due to work laptop)

Hm @full-student-91825 I've spent 20 minutes writing instructions and I'm realizing at least the pre-work is a little bit more involved than I expected. Would you be interested in if I do the pre-work, and then write instructions for how to finish out the change?

@hundreds-father-404 sounds good, though I might not get to the task until next week 😕

fyi https://github.com/pantsbuild/pants/pull/15186. i still need to update the rest of the call sites