Skip to content

Conversation

2vg
Copy link
Contributor

@2vg 2vg commented Apr 8, 2018

add: Nim's web framework "mofuw".
I pray that no problems will occur this time lol.

@mention-bot
Copy link

Thanks @2vg for contributing to The Framework Benchmarks! @TechEmpower, code you've worked on has been modified. If you have the chance, please review. If you wish to unsubscribe from these notices, please open a Pull Request with the commit message [ci skip] and your github name added to the userBlacklist array in the .mention-bot file.

@2vg
Copy link
Contributor Author

2vg commented Apr 8, 2018

I do not know the cause, but I pass the first test, but the same problem as before is happening the second time.
Do not merge this pull request yet as it builds in debug mode to investigate the problem.

@2vg
Copy link
Contributor Author

2vg commented Apr 8, 2018

... I understood the cause
The time update function for the Date header is executed for after 1 second the server is started, but the problem occurred because the benchmark was executed earlier than updating the first time at time.
I fixed it.

@2vg
Copy link
Contributor Author

2vg commented Apr 8, 2018

yeee, all ok. It can be merged !!!

@NateBrady23
Copy link
Member

Awesome!

@NateBrady23 NateBrady23 merged commit 0701cfc into TechEmpower:master Apr 8, 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.

3 participants