-
-
Notifications
You must be signed in to change notification settings - Fork 626
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Issue/10944 pep561 py typed 2 #12842
Conversation
Signed-off-by: Andreas Stenius <andreas.stenius@svenskaspel.se> [ci skip-rust] [ci skip-build-wheels]
Signed-off-by: Andreas Stenius <andreas.stenius@svenskaspel.se> # Rust tests and lints will be skipped. Delete if not intended. [ci skip-rust] # Building wheels and fs_util will be skipped. Delete if not intended. [ci skip-build-wheels]
Added the cherry-pick label, as I think this is a rather unobtrusive change, but adds a much needed feature in order to write pants plugins effectively. Feel free to disagree ;) |
Signed-off-by: Andreas Stenius <andreas.stenius@svenskaspel.se> # Rust tests and lints will be skipped. Delete if not intended. [ci skip-rust] # Building wheels and fs_util will be skipped. Delete if not intended. [ci skip-build-wheels]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great: thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Woot, thanks!
…EP-561 (pantsbuild#12842) Closes pantsbuild#10944 Supersedes pantsbuild#12330, pantsbuild#12751 This adds `py.typed` to `pantsbuild.pants` and `pantsbuild.pants.testutil`, with integration test verifying that typecheck works as expected for both distributions. I've verified that the tests fail as expected when excluding the dependency on the `py_typed` resource. (cherry picked from commit 4f437a4) # Rust tests and lints will be skipped. Delete if not intended. [ci skip-rust] # Building wheels and fs_util will be skipped. Delete if not intended. [ci skip-build-wheels]
Closes #10944 Supersedes #12330, #12751
This adds
py.typed
topantsbuild.pants
andpantsbuild.pants.testutil
, with integration test verifying that typecheck works as expected for both distributions.I've verified that the tests fail as expected when excluding the dependency on the
py_typed
resource.