Skip to content

Conversation

@prog1dev
Copy link
Contributor

@prog1dev prog1dev commented Mar 27, 2018

Original pr #18357

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

@MylesBorins

@nodejs-github-bot nodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. dont-land-on-v4.x whatwg-url Issues and PRs related to the WHATWG URL implementation. labels Mar 27, 2018
@prog1dev
Copy link
Contributor Author

prog1dev commented Mar 29, 2018

Something went wrong 😄
+848 −38,000

@TimothyGu
Copy link
Member

@prog1dev It’s probably because there have been force-pushes to the v8.x-staging branch. A careful rebase should fix the problem.

Fixes: nodejs#18302

PR-URL: nodejs#18357
Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
@prog1dev prog1dev force-pushed the backport-18357-to-v6.x branch from f067901 to af2c41e Compare April 1, 2018 12:42
@MylesBorins
Copy link
Contributor

MylesBorins pushed a commit that referenced this pull request Apr 13, 2018
Fixes: #18302

Backport-PR-URL: #19639
PR-URL: #18357
Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
@MylesBorins
Copy link
Contributor

landed in 0ca2dad

@prog1dev prog1dev deleted the backport-18357-to-v6.x branch April 13, 2018 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++ Issues and PRs that require attention from people who are familiar with C++. whatwg-url Issues and PRs related to the WHATWG URL implementation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants