Skip to content

Fix E501 in core utils - #5300

Closed
saitcakmak wants to merge 1 commit into
meta:mainfrom
saitcakmak:export-D125714181
Closed

saitcakmak wants to merge 1 commit into
meta:mainfrom
saitcakmak:export-D125714181

Conversation

@saitcakmak

Copy link
Copy Markdown
Contributor

Summary:

What Changed

Move the existing Pyrefly suppression above the keyword argument in _sort_trials, fixing the 113-character line without changing runtime behavior.

Differential Revision: D125714181

Summary:
## What Changed
Move the existing Pyrefly suppression above the keyword argument in `_sort_trials`, fixing the 113-character line without changing runtime behavior.

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

meta-codesync Bot commented Oct 16, 2026

Copy link
Copy Markdown

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

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

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

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5300   +/-   ##
=======================================
  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.

@meta-codesync meta-codesync Bot closed this in 7017487 Oct 16, 2026
@meta-codesync meta-codesync Bot added the Merged label Oct 16, 2026
@meta-codesync

meta-codesync Bot commented Oct 16, 2026

Copy link
Copy Markdown

This pull request has been merged in 7017487.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants