-
Notifications
You must be signed in to change notification settings - Fork 1k
Updated formatting #7135
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
Updated formatting #7135
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #7135 +/- ##
==========================================
- Coverage 98.51% 98.50% -0.01%
==========================================
Files 81 81
Lines 15013 15015 +2
==========================================
+ Hits 14790 14791 +1
- Misses 223 224 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
No obvious timing issues in HEAD=fwriteReformat Generated via commit 6c50dbc Download link for the artifact containing the test results: ↓ atime-results.zip
|
MichaelChirico
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! What was done this round to avoid the need for follow-up PRs? It makes the git history cleaner if we can avoid multiple "style-only" edits
|
@MichaelChirico I thought it was better to separate style out. I can add more in on top, in future PRs. Up to you. |
|
I think this one is simpler, so that won't be needed. I'll make sure to include other improvements with future PRs |

Hopefully no conflicts?
Once this is merged I'll get more aggressive with #6968 and #6971