Skip to content

Suppress type errors for Pyre upgrade (landing group 4) (#5298) - #5298

Closed
maggiemoss wants to merge 1 commit into
facebook:mainfrom
maggiemoss:export-D123975487
Closed

maggiemoss wants to merge 1 commit into
facebook:mainfrom
maggiemoss:export-D123975487

Conversation

@maggiemoss

@maggiemoss maggiemoss commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

Summary:
X-link: pytorch/FBGEMM#6417

X-link: https://github.com/facebookresearch/FBGEMM/pull/3278

This diff was automatically generated by the Pyre per-target upgrade tool.

It adds # pyrefly: ignore comments for type errors that an upcoming Pyrefly version will report. It changes comments only; it does not change Python runtime behavior.

Landing group: e5cd8bb1932882651ede8b1556fb5d4edfc7db11f903fed396e610d810c715d8 (attempt 1, 20 batches, 54 files)
Merged patch SHA-256: 4d1b81b93caadf1bc3a3bdf05e1317ce2186abe77754c5a430539a158db52e59

Full fbcode Pyrefly per-target upgrade using D123902372 typed Buck uquery scheduler; target commit af8bbadc839ed3725786ccbc573c25769d4a144b.

#pyreupgrade

Differential Revision: D123975487

@meta-cla meta-cla Bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Oct 7, 2026
@meta-codesync

meta-codesync Bot commented Oct 7, 2026

Copy link
Copy Markdown

@maggiemoss has exported this pull request. If you are a Meta employee, you can view the originating Diff in D123975487.

Summary:
X-link: pytorch/FBGEMM#6417

X-link: https://github.com/facebookresearch/FBGEMM/pull/3278


This diff was automatically generated by the Pyre per-target upgrade tool.

It adds `# pyrefly: ignore` comments for type errors that an upcoming Pyrefly version will report. It changes comments only; it does not change Python runtime behavior.

Landing group: `e5cd8bb1932882651ede8b1556fb5d4edfc7db11f903fed396e610d810c715d8` (attempt 1, 20 batches, 54 files)
Merged patch SHA-256: `4d1b81b93caadf1bc3a3bdf05e1317ce2186abe77754c5a430539a158db52e59`

Full fbcode Pyrefly per-target upgrade using D123902372 typed Buck uquery scheduler; target commit af8bbadc839ed3725786ccbc573c25769d4a144b.

#pyreupgrade

Differential Revision: D123975487
@meta-codesync meta-codesync Bot changed the title Suppress type errors for Pyre upgrade (landing group 4) Suppress type errors for Pyre upgrade (landing group 4) (#5298) Oct 7, 2026
maggiemoss pushed a commit to maggiemoss/FBGEMM that referenced this pull request Oct 7, 2026
Summary:

X-link: https://github.com/facebookresearch/FBGEMM/pull/3278

X-link: facebook/Ax#5298

This diff was automatically generated by the Pyre per-target upgrade tool.

It adds `# pyrefly: ignore` comments for type errors that an upcoming Pyrefly version will report. It changes comments only; it does not change Python runtime behavior.

Landing group: `e5cd8bb1932882651ede8b1556fb5d4edfc7db11f903fed396e610d810c715d8` (attempt 1, 20 batches, 54 files)
Merged patch SHA-256: `4d1b81b93caadf1bc3a3bdf05e1317ce2186abe77754c5a430539a158db52e59`

Full fbcode Pyrefly per-target upgrade using D123902372 typed Buck uquery scheduler; target commit af8bbadc839ed3725786ccbc573c25769d4a144b.

#pyreupgrade

Differential Revision: D123975487
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.59%. Comparing base (f4e58e0) to head (4d98cab).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5298   +/-   ##
=======================================
  Coverage   96.59%   96.59%           
=======================================
  Files         621      621           
  Lines       70966    70966           
=======================================
  Hits        68550    68550           
  Misses       2416     2416           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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. meta-exported

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants