Repository navigation
url: export URLSearchParams alongside URL #10761
Description
Activity
- added a commit that references this issue
on Jan 14, 2017 @targos @TimothyGu This commit doesn't seem to to have the necessary metadata in the commit message?
Oh I didn't mean to push this commit. This was a mistake.
@nodejs/ctc the 10 minutes to force push are long gone but can we still remove it?Maybe just revert and reland it?
I guess that harm is done anyway (regarding tooling). I added the metadata as a comment in the commit.
Reacted by Anna Henningsen and Gibson Fahnestock@joyeecheung the way we handle it is within the first ten minutes after a commit, and only if no other commits have landed since, a commit can be edited or amended to fix metadata with a force push. That is the only window of time given to correct a commit without requiring a full revert pr that would go through the normal review process.
@jasnell What I meant was a revert PR followed by a reland PR though. The release team has been notified via IRC so I think it's OK to leave it as-is.
It's typically not worth the additional clutter that would cause simply to update the metadata :-)
- added 4 commits that reference this issue
on Jan 25, 2017
The
URLSearchParamsobject is useful on it's own. Browsers allow it to be created and used independently, so should we.Refs: #10454