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
cloudflare-cli: init v3.2.0 #48030
cloudflare-cli: init v3.2.0 #48030
Conversation
6e253e7
to
54d305c
Compare
@@ -1872,7 +1872,7 @@ let | |||
packageName = "strip-ansi"; | |||
version = "3.0.1"; | |||
src = fetchurl { | |||
url = "https://registry.npmjs.org/strip-ansi/-/strip-ansi-3.0.1.tgz"; | |||
url = "http://registry.npmjs.org/strip-ansi/-/strip-ansi-3.0.1.tgz"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not versed in Node packaging but this feels unnecessary.
@@ -1872,7 +1872,7 @@ let | |||
packageName = "strip-ansi"; | |||
version = "3.0.1"; | |||
src = fetchurl { | |||
url = "https://registry.npmjs.org/strip-ansi/-/strip-ansi-3.0.1.tgz"; | |||
url = "http://registry.npmjs.org/strip-ansi/-/strip-ansi-3.0.1.tgz"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same here.
Thank you for the review @c0bw3b. I tend not to edit things that have been auto-generated (in this case, the But, obviously I don't think the cases you've found are good changes. @svanderburg Can you weigh in on this? Is there a reason that these would be getting switched to Thanks all. |
@@ -4392,7 +4401,7 @@ let | |||
packageName = "bl"; | |||
version = "0.8.2"; | |||
src = fetchurl { | |||
url = "https://registry.npmjs.org/bl/-/bl-0.8.2.tgz"; | |||
url = "http://registry.npmjs.org/bl/-/bl-0.8.2.tgz"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
and here
@@ -4401,7 +4410,7 @@ let | |||
packageName = "bl"; | |||
version = "1.0.3"; | |||
src = fetchurl { | |||
url = "https://registry.npmjs.org/bl/-/bl-1.0.3.tgz"; | |||
url = "http://registry.npmjs.org/bl/-/bl-1.0.3.tgz"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
and here... (and a number after this too.)
NOTE to self: per this NixCon talk (https://youtu.be/vv4d42vbICw?t=41m7s) consider using pnpm2nix: https://github.com/adisbladis/pnpm2nix |
I don't use this, don't want to maintain it, it doesn't merge, etc. Closing... |
Motivation for this change
Add
cfcli
(cloudflare-cli)Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)