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

Fix longitudinal benchmark #5514

Merged
merged 1 commit into from
Sep 1, 2023
Merged

Fix longitudinal benchmark #5514

merged 1 commit into from
Sep 1, 2023

Conversation

zliu41
Copy link
Member

@zliu41 zliu41 commented Sep 1, 2023

This was broken by #5371

"The function that receives the environment must use lazy pattern matching": https://hackage.haskell.org/package/criterion-1.6.3.0/docs/Criterion.html#v:env

Otherwise

validation: Criterion atttempted to retrieve a non-existent environment!
	Perhaps you forgot to use lazy pattern matching in a function which
	constructs benchmarks from an environment?
	(see the documentation for `env` for details)

@zliu41 zliu41 enabled auto-merge (squash) September 1, 2023 20:25
@zliu41 zliu41 added the No Changelog Required Add this to skip the Changelog Check label Sep 1, 2023
@zliu41 zliu41 merged commit 47b6b7e into master Sep 1, 2023
7 of 8 checks passed
@zliu41 zliu41 deleted the zliu41/long branch September 1, 2023 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No Changelog Required Add this to skip the Changelog Check
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant