Skip to content
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

Add compile time to PR summary #6540

Closed
Tracked by #5070
benesjan opened this issue May 20, 2024 · 1 comment
Closed
Tracked by #5070

Add compile time to PR summary #6540

benesjan opened this issue May 20, 2024 · 1 comment

Comments

@benesjan
Copy link
Contributor

Recently there was a huge noir circuits compile time increase due to inefficient code (fix here).

This is very easy to miss when you don't see the time difference and hence should be added to the PR summary.

@benesjan
Copy link
Contributor Author

benesjan commented Aug 9, 2024

Not planned

@benesjan benesjan closed this as completed Aug 9, 2024
@github-project-automation github-project-automation bot moved this from Todo to Done in A3 Aug 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

No branches or pull requests

1 participant