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

Compile URI using minim #85

Closed
wants to merge 3 commits into from
Closed

Conversation

honzajavorek
Copy link
Contributor

@honzajavorek honzajavorek commented Jul 26, 2017

This PR makes sure everything in the compile-uri directory introduced in #84 uses minim instead of raw refract.

@honzajavorek
Copy link
Contributor Author

I'm getting timeouts in the tests 😞

@michalholasek
Copy link

@honzajavorek So we are running into minim performance problems once again I suppose?

@honzajavorek
Copy link
Contributor Author

Depends. Either that or it's overuse of serialize/deserialize (which should be temporary though). Let's continue with this after investigating why the performance dropped in #87.

@honzajavorek
Copy link
Contributor Author

honzajavorek commented Aug 18, 2017

When resumed, this PR needs to incorporate what happened in #94 and to fix several things introduced in #94:

@honzajavorek
Copy link
Contributor Author

Superseded by #97.

@honzajavorek honzajavorek deleted the honzajavorek/compile-uri-minim branch August 18, 2017 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants