-
Notifications
You must be signed in to change notification settings - Fork 268
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
fix pyre targets #508
fix pyre targets #508
Conversation
This pull request was exported from Phabricator. Differential Revision: D48478829 |
Codecov Report
@@ Coverage Diff @@
## master #508 +/- ##
=======================================
Coverage 86.95% 86.95%
=======================================
Files 106 106
Lines 8407 8407
=======================================
Hits 7310 7310
Misses 1097 1097
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
This pull request was exported from Phabricator. Differential Revision: D48478829 |
Summary: Pull Request resolved: pytorch#508 # Context `pyre` is not configured properly for `torchtnt`. The config is in `torchtnt/.pyre_configuration.local` and the coverage is very partial # This diff 1. Fix `torchtnt/.pyre_configuration.local` 2. Codemod and add `pyre-fixme` to any pyre errors as a result `pyre --output=json check | pyre-upgrade fixme` 3. Fix any `lints` as a result 4. Fix the `pyre` issues that `lint` resurfaced Reviewed By: ananthsub Differential Revision: D48478829 fbshipit-source-id: d5ec136b7a92c8fa5fb0a178b9a318b5295ed62e
4e4ea76
to
032f6b4
Compare
Summary: Pull Request resolved: pytorch#508 # Context `pyre` is not configured properly for `torchtnt`. The config is in `torchtnt/.pyre_configuration.local` and the coverage is very partial # This diff 1. Fix `torchtnt/.pyre_configuration.local` 2. Codemod and add `pyre-fixme` to any pyre errors as a result `pyre --output=json check | pyre-upgrade fixme` 3. Fix any `lints` as a result 4. Fix the `pyre` issues that `lint` resurfaced Reviewed By: ananthsub Differential Revision: D48478829 fbshipit-source-id: fdcb7d5fa1898a07a68dd97a29242e3596bb2907
This pull request was exported from Phabricator. Differential Revision: D48478829 |
032f6b4
to
398f1e5
Compare
Summary:
Context
pyre
is not configured properly fortorchtnt
. The config is intorchtnt/.pyre_configuration.local
and the coverage is very partialThis diff
torchtnt/.pyre_configuration.local
pyre-fixme
to any pyre errors as a resultpyre --output=json check | pyre-upgrade fixme
lints
as a resultpyre
issues thatlint
resurfacedDifferential Revision: D48478829