Skip to content
This repository was archived by the owner on Mar 24, 2026. It is now read-only.

Haskell/Yesod tests: Code reorg#2258

Merged
knewmanTE merged 5 commits intoTechEmpower:masterfrom
vacationlabs:round-13
Sep 20, 2016
Merged

Haskell/Yesod tests: Code reorg#2258
knewmanTE merged 5 commits intoTechEmpower:masterfrom
vacationlabs:round-13

Conversation

@saurabhnanda
Copy link
Copy Markdown
Contributor

@saurabhnanda
Copy link
Copy Markdown
Contributor Author

@msmith-techempower @nbrady-techempower please review this PR. The Yesod tests are passing Travis CI. Don't know why the other stuff is failing -- I didn't touch anything else in my branch.

In Travis CI, will Yesod show up as one entry or two entries - yesod-mysql and yesod-postgres?

@msmith-techempower
Copy link
Copy Markdown
Member

@saurabhnanda Agreed, Travis-CI passes. Let's get this merged into round-14 since it just code reorganization.

@saurabhnanda
Copy link
Copy Markdown
Contributor Author

Okay. When is the next round of preliminary (or final) results out?

On 6 Sep 2016 11:00 pm, "Mike Smith" notifications@github.com wrote:

@saurabhnanda https://github.com/saurabhnanda Agreed, Travis-CI passes.
Let's get this merged into round-14 since it just code reorganization.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#2258 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABu0VCpwF4gqcFWMABYVXncXz0tCPlyks5qnZrdgaJpZM4J0U7S
.

@knewmanTE
Copy link
Copy Markdown
Contributor

@saurabhnanda would you mind closing this PR and re-opening it against the round-14 branch?

re: the next results, we're going to run a second preview round this weekend on our high-performance hardware setup with the plan of publishing the results next week. The Round 13 Final run should follow shortly after across both of our test environments.

@knewmanTE knewmanTE added this to the Round 13 milestone Sep 20, 2016
@knewmanTE knewmanTE merged commit 1ba26b0 into TechEmpower:master Sep 20, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants