try to get this to work. #6
Conversation
|
this is just for reference, feel free to close this PR at any time. |
| @@ -49,6 +49,7 @@ inits = true | |||
|
|
|||
| [black] | |||
| version = "black==20.8b1" | |||
| extra_requirements = ["regex>=2021.3.17"] | |||
There was a problem hiding this comment.
this is the config we have in our internal repo... it works.... If I really think about it was probably added to override the pants default setting here, which I think is setuptools (or at least used to be) which caused issues in the past.
There was a problem hiding this comment.
I double checked, there is no longer a need for this (at least with recent pants versions)
| # Works around bad `-arch arm64` flag embedded in Xcode 12.x Python interpreters on intel | ||
| # machines. See: https://github.com/giampaolo/psutil/issues/1832 | ||
| # On macos do: export ARCHFLAGS=-arch x86_64 | ||
| # see: https://github.com/pantsbuild/pants/pull/11733 | ||
| "ARCHFLAGS", |
There was a problem hiding this comment.
When I hit this issue, I switched over to my MacPorts-installed copies of python. Good to know there's a solution :)
This won't affect any other platforms will it?
There was a problem hiding this comment.
this option just tells pants to pass this env variable (if defined) to subprocesses. it doesn't actually define it.
in CI, pants defines it for mac os builds, see pantsbuild/pants#11733
There was a problem hiding this comment.
and there is a thread discussion in pants slack: https://pantsbuild.slack.com/archives/C0D7TNJHL/p1617914862144100
|
I've got pylint running now. Hooray! Thanks for your help in here. I'll close this. |
linter pass (except for an issue w/ pylint run not having access to some json file)
@cognifloyd I hope this helps!