Skip to content
This repository has been archived by the owner on Jan 9, 2024. It is now read-only.

chore(package): update request to version 2.79.0 #245

Merged
merged 1 commit into from
Jan 28, 2017

Conversation

josephfrazier
Copy link
Contributor

This avoids the following deprecation warning upon npm install:

npm WARN deprecated [email protected]: use uuid module instead

request switched from node-uuid to uuid in version 2.79.0:
request/request@263e16e

This avoids the following deprecation warning upon `npm install`:

    npm WARN deprecated [email protected]: use uuid module instead

`request` switched from `node-uuid` to `uuid` in version 2.79.0:
request/request@263e16e
@kumar303
Copy link
Contributor

Thanks

@kumar303 kumar303 merged commit 097dfed into mozilla:master Jan 28, 2017
@josephfrazier josephfrazier deleted the request-2.79.0 branch January 28, 2017 02:22
@josephfrazier
Copy link
Contributor Author

Sure thing, thanks for the quick merge! Any chance you could release version 0.2.1? It'd be nice to get web-ext upgraded.

josephfrazier added a commit to josephfrazier/browser-extension that referenced this pull request Jan 28, 2017
This avoids the following deprecation warning upon `npm install`:

    npm WARN deprecated [email protected]: use uuid module instead

`web-ext` still uses `node-uuid` indirectly (via `sign-addon` -> `request`),
but I'm working on fixing that too. See mozilla/sign-addon#245
@kumar303
Copy link
Contributor

Sure, I can release sign-addon on Monday. We'll cut a web-ext release after this gets merged: mozilla/web-ext#753

stefanbuck pushed a commit to OctoLinker/OctoLinker that referenced this pull request Jan 28, 2017
This avoids the following deprecation warning upon `npm install`:

    npm WARN deprecated [email protected]: use uuid module instead

`web-ext` still uses `node-uuid` indirectly (via `sign-addon` -> `request`),
but I'm working on fixing that too. See mozilla/sign-addon#245
@josephfrazier
Copy link
Contributor Author

Great to hear, thanks again!

@caitmuenster
Copy link

Thanks, @josephfrazier! Your contribution has been added to our recognition wiki.

Welcome onboard!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants