Skip to content

Conversation

Razican
Copy link
Member

@Razican Razican commented Oct 23, 2022

This is the first change towards v4, which ideally should include #8, #80 and only showing "changing" benchmarks by default.

This fixes #8, but it has not been thoroughly tested, so I would like to test this in Boa before merging.

@finnbear
Copy link

Thanks for implementing these changes! A minor suggestion is to move the listComments query inside the try { } block. Otherwise, its failure (due to an HTTP/permissions error) causes the fallback table to not be printed.

@chenyan2002
Copy link

Any update on this PR?

@leon0399
Copy link

Would love to see this merged!

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.

[Feature Request] Update the benchmark comment, don't create a new one
4 participants