Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

More simplification #386

Merged
merged 5 commits into from
Oct 5, 2019
Merged

More simplification #386

merged 5 commits into from
Oct 5, 2019

Conversation

GrahamTheCoder
Copy link
Member

Problem

Currently the simplification is haphazard due to the order that files are converted.

Solution

This introduces a phase to create a fully converted project before the simplification phase.
Due to the consistency, it's now possible to parallelize and maintain determinism.

There's a bit of leading whitespace introduced in a handful of cases. Didn't seem important enough to try fixing.

  • At least one test covering the code changed (see recharacterization results)
  • All tests pass

@GrahamTheCoder GrahamTheCoder merged commit b8a5925 into master Oct 5, 2019
@BrianFreemanAtlanta BrianFreemanAtlanta mentioned this pull request Nov 9, 2019
1 task
@GrahamTheCoder GrahamTheCoder deleted the more-simplification branch February 28, 2021 19:46
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant