Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

url: significantly improve the performance of the url module #1561

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.

Sign up for GitHub

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

Jump to bottom
Closed
petkaantonov wants to merge 1 commit into from
Closed

url: significantly improve the performance of the url module #1561

petkaantonov wants to merge 1 commit into from

Conversation

@petkaantonov
Copy link
Contributor

@petkaantonov petkaantonov commented Apr 29, 2015

New PR to target master

This was referenced Apr 29, 2015
Closed
Closed
@petkaantonov petkaantonov added the semver-major PRs that contain breaking changes and should be released in the next major version. label Apr 29, 2015
@petkaantonov petkaantonov added this to the 2.0.0 milestone Apr 29, 2015
@Fishrock123
Copy link
Contributor

Fishrock123 commented Apr 29, 2015

CI: https://jenkins-iojs.nodesource.com/view/iojs/job/iojs+any-pr+multi/636/

@piscisaureus
Copy link
Contributor

piscisaureus commented Apr 29, 2015

@petkaantonov Is this ready to land in your mind?

@petkaantonov
Copy link
Contributor Author

petkaantonov commented Apr 29, 2015

@piscisaureus yes although @domenic might want to check it out

@domenic
Copy link
Contributor

domenic commented Apr 29, 2015

LGTM!

@jbergstroem
Copy link
Member

jbergstroem commented Apr 30, 2015

Very happy about getting the WhatWG tests in. Great job both @petkaantonov and all diligent reviewers.

@Fishrock123
Copy link
Contributor

Fishrock123 commented May 2, 2015

Original PR for posterity: #933

@domenic
Copy link
Contributor

domenic commented May 2, 2015

What is this waiting on before it can be merged? Nothing, right?

petkaantonov added a commit that referenced this pull request May 2, 2015
@petkaantonov @rvagg
url: significantly improve the performance of the url module
3fd7fc4 
(landed by @rvagg)

PR-URL: #1561
Reviewed-By: Domenic Denicola <domenic@domenicdenicola.com>
Reviewed-By: Rod Vagg <rod@vagg.org>
@rvagg
Copy link
Member

rvagg commented May 2, 2015

took the liberty of landing this, I hope you don't mind @petkaantonov. 3fd7fc4

@rvagg rvagg closed this May 2, 2015
This was referenced May 2, 2015
Closed
Closed
@isaacs isaacs mentioned this pull request May 3, 2015
Merged
@othiym23 othiym23 mentioned this pull request May 5, 2015
Closed
@Fishrock123 Fishrock123 mentioned this pull request Jun 10, 2015
Closed
@Fishrock123 Fishrock123 mentioned this pull request Jul 30, 2015
Closed
@STRML STRML mentioned this pull request Aug 5, 2015
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

semver-major PRs that contain breaking changes and should be released in the next major version.

Projects

None yet

Milestone

2.0.0

Development

Successfully merging this pull request may close these issues.

6 participants

@petkaantonov @Fishrock123 @piscisaureus @domenic @jbergstroem @rvagg
Morty Proxy This is a proxified and sanitized view of the page, visit original site.