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

Simplify handle_mult using Pvar pattern #423

Closed
wants to merge 14 commits into from
Closed

Simplify handle_mult using Pvar pattern #423

wants to merge 14 commits into from

Conversation

myreen
Copy link
Contributor

@myreen myreen commented Jan 8, 2018

No description provided.

@xrchz
Copy link
Member

xrchz commented Jan 8, 2018

regression test is failing - reopen on fix

@xrchz xrchz closed this Jan 8, 2018
@myreen myreen reopened this Jan 10, 2018
@xrchz
Copy link
Member

xrchz commented Jan 10, 2018

I don't know why 146 failed. Running the regression test again to see if it fails again.

@xrchz
Copy link
Member

xrchz commented Jan 11, 2018

Is this still work-in-progress, or ready for merge?

@myreen
Copy link
Contributor Author

myreen commented Jan 11, 2018 via email

@xrchz
Copy link
Member

xrchz commented Jan 12, 2018

This is consistently failing the regression test in a mysterious way. I'm closing it for now - reopen when/if someone can confirm that the failing directory actually succeeds. I can produce a tarball of the regression test worker's state if desired.

@xrchz xrchz closed this Jan 12, 2018
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.

None yet

3 participants