-
Notifications
You must be signed in to change notification settings - Fork 4
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
Update tutorials formatting #220
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #220 +/- ##
=======================================
Coverage 92.47% 92.47%
=======================================
Files 40 40
Lines 877 877
=======================================
Hits 811 811
Misses 66 66
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Confirming this does not (and is not intended to) close #159, correct? It does the formatting, but doesn't set it up in |
@dylanhmorris That's correct. I only linked to #159 to explain why it was not done automatically. |
I am confused. Is this formatting done automatically, or no? @damonbayer @dylanhmorris |
It can be done semi-automatically in VS Code. However, it only works on a cell-by-cell basis, so you have to go through each cell and issue a formatting command (or re-save the file if you have format on save enabled). This is what I did to create the changes in this PR. As far as I know, there is no way to format an entire |
Anyone willing to approve this @dylanhmorris @sbidari @AFg6K7h4fhy2 ? |
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.
LGTM
See #159.