Skip to content
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

When complaining about a formatting error, say what the error is #363

Closed
wants to merge 1 commit into from

Conversation

shish
Copy link
Contributor

@shish shish commented Mar 26, 2024

When complaining about a formatting error, say what the error is

I'm getting an error which only appears on python 3.7 and I can't repro it locally, and CI won't tell me what the error is x_x

This was referenced Mar 26, 2024
@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Mar 26, 2024
I'm getting an error which _only_ appears on python 3.7 and I can't repro it locally, and CI won't tell me what the error is x_x
@facebook-github-bot
Copy link

@shish has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

facebook-github-bot pushed a commit that referenced this pull request Mar 26, 2024
Summary:
Fix type signatures in sample module

mypy is complaining about this and failing all of the builds, and there's no comment saying "this mistake is intentional", so I assume it is an accident

 ---
[//]: # (BEGIN SAPLING FOOTER)
Stack created with [Sapling](https://sapling-scm.com). Best reviewed with [ReviewStack](https://reviewstack.dev/facebook/TestSlide/pull/361).
* #362
* #363
* __->__ #361

Pull Request resolved: #361

Differential Revision: D55365462

Pulled By: shish

fbshipit-source-id: 61cac16eeb3bac081ce95fe7fde3e6f9b4d4b693
@facebook-github-bot
Copy link

@shish merged this pull request in 86a7124.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Do not delete this pull request or issue due to inactivity. Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants