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

[BUGFIX] Absorb breakage in benchmark-env instead of benchmark #1220

Merged
merged 1 commit into from
Dec 2, 2020

Conversation

pzuraq
Copy link
Member

@pzuraq pzuraq commented Dec 2, 2020

Some previous PRs made changes to the benchmark, but this made the benchmark
itself incompatible with previous versions of Glimmer. This is something that
is valuable to be able to test, so for now are going to keep absorbing all
breakage in the benchmark-env.

Some previous PRs made changes to the benchmark, but this made the benchmark
itself incompatible with previous versions of Glimmer. This is something that
is valuable to be able to test, so for now are going to keep absorbing all
breakage in the benchmark-env.
@pzuraq pzuraq force-pushed the bugfix/absorb-breakage-in-benchmark-env branch from a50e276 to 40ea47f Compare December 2, 2020 21:40
@pzuraq
Copy link
Member Author

pzuraq commented Dec 2, 2020

Krausest will fail for this change, since the previous commit depended on setComponentTemplate being called within the benchmark and not the benchmark-env. I've tested it locally, and the experiment does serve succesfully.

@pzuraq pzuraq merged commit 3f4c054 into master Dec 2, 2020
@pzuraq pzuraq deleted the bugfix/absorb-breakage-in-benchmark-env branch December 2, 2020 22:59
@pzuraq pzuraq added bug internal and removed bug labels Dec 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant