Skip to content

Conversation

dolio
Copy link
Contributor

@dolio dolio commented Sep 2, 2025

This updates a transcript that was missed in the profiling PR. It's just the output of the help for the new profiling commands.

@dolio dolio requested a review from aryairani September 2, 2025 19:07
Copy link
Contributor

@aryairani aryairani left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i'm surprised it didn't complain in ci

@dolio
Copy link
Contributor Author

dolio commented Sep 2, 2025

It did, but the ormolu commit lets you merge anyway, I think. I don't know if there's a way to make those not count, but it'd probably be worthwhile.

@aryairani
Copy link
Contributor

It did, but the ormolu commit lets you merge anyway, I think. I don't know if there's a way to make those not count, but it'd probably be worthwhile.

Yeah I started on something like that, but haven't committed it yet (not sure if it will work either).

@aryairani aryairani merged commit 1ce29c7 into trunk Sep 3, 2025
31 checks passed
@aryairani aryairani deleted the topic/profiling branch September 3, 2025 13:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants