Skip to content

Conversation

@rsokl
Copy link
Collaborator

@rsokl rsokl commented Oct 24, 2025

Stacked on top of #3089

The majority of changes are trivial test updates. An additional hack is needed to continue to support LazyCompletionHelp -- temporarily disabling argparse.ArgumentParser._check_help -- but this is pretty benign and we add a test to ensure that things still work.

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Oct 24, 2025
@rsokl rsokl changed the base branch from main to drop-python-3.8-support October 24, 2025 19:04
@rsokl rsokl marked this pull request as ready for review October 26, 2025 16:54
@rsokl rsokl requested a review from Jasha10 October 26, 2025 17:02
@rsokl rsokl marked this pull request as draft October 26, 2025 17:02
@rsokl rsokl marked this pull request as ready for review October 26, 2025 17:44
select = E,F,W,C
copyright-regexp=Copyright \(c\) Facebook, Inc. and its affiliates. All Rights Reserved
ignore=W503,E203,E231,E701,E704
ignore=W503,E203,E231,E241,E701,E704
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why ignore E241?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was hitting commit-blocking false positives on some literal string patterns. Black would already reformat any code that would have actually violated this

Base automatically changed from drop-python-3.8-support to main October 27, 2025 13:21
@rsokl rsokl force-pushed the add-future-support branch from fbd76f9 to 71c6ea2 Compare October 27, 2025 13:25
@rsokl rsokl merged commit 7f1d096 into main Oct 28, 2025
83 of 109 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants