Skip to content
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

Use https when possible. #7

Merged
merged 1 commit into from
Jul 23, 2018
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
npm(1) -- a JavaScript package manager
==============================

[![Build Status](https://img.shields.io/travis/npm/npm/latest.svg)](https://travis-ci.org/npm/npm)
[![Build Status](https://img.shields.io/travis/npm/cli/latest.svg)](https://travis-ci.org/npm/cli)

## SYNOPSIS

Expand Down
4 changes: 2 additions & 2 deletions doc/cli/npm-hook.md
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ $ npm hook rm id-deadbeef
## DESCRIPTION

Allows you to manage [npm
hooks](http://blog.npmjs.org/post/145260155635/introducing-hooks-get-notifications-of-npm),
hooks](https://blog.npmjs.org/post/145260155635/introducing-hooks-get-notifications-of-npm),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes in this file are ok. I'm alright keeping these.

including adding, removing, listing, and updating.

Hooks allow you to configure URL endpoints that will be notified whenever a
Expand All @@ -69,4 +69,4 @@ request came from your own configured hook.

## SEE ALSO

* ["Introducing Hooks" blog post](http://blog.npmjs.org/post/145260155635/introducing-hooks-get-notifications-of-npm)
* ["Introducing Hooks" blog post](https://blog.npmjs.org/post/145260155635/introducing-hooks-get-notifications-of-npm)
4 changes: 2 additions & 2 deletions doc/cli/npm-run-script.md
Original file line number Diff line number Diff line change
Expand Up @@ -15,9 +15,9 @@ used by the test, start, restart, and stop commands, but can be called
directly, as well. When the scripts in the package are printed out, they're
separated into lifecycle (test, start, restart) and directly-run scripts.

As of [`npm@2.0.0`](http://blog.npmjs.org/post/98131109725/npm-2-0-0), you can
As of [`npm@2.0.0`](https://blog.npmjs.org/post/98131109725/npm-2-0-0), you can
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ditto above: this file is ok to keep the changes for.

use custom arguments when executing scripts. The special option `--` is used by
[getopt](http://goo.gl/KxMmtG) to delimit the end of the options. npm will pass
[getopt](https://goo.gl/KxMmtG) to delimit the end of the options. npm will pass
all the arguments after the `--` directly to your script:

npm run test -- --grep="pattern"
Expand Down
2 changes: 1 addition & 1 deletion doc/cli/npm-start.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ This runs an arbitrary command specified in the package's `"start"` property of
its `"scripts"` object. If no `"start"` property is specified on the
`"scripts"` object, it will run `node server.js`.

As of [`npm@2.0.0`](http://blog.npmjs.org/post/98131109725/npm-2-0-0), you can
As of [`npm@2.0.0`](https://blog.npmjs.org/post/98131109725/npm-2-0-0), you can
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This one's ok too.

use custom arguments when executing scripts. Refer to npm-run-script(1) for
more details.

Expand Down
2 changes: 1 addition & 1 deletion doc/cli/npm.md
Original file line number Diff line number Diff line change
Expand Up @@ -140,7 +140,7 @@ reproduction to report.

[Isaac Z. Schlueter](http://blog.izs.me/) ::
[isaacs](https://github.com/isaacs/) ::
[@izs](http://twitter.com/izs) ::
[@izs](https://twitter.com/izs) ::
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok!

<i@izs.me>

## SEE ALSO
Expand Down
4 changes: 2 additions & 2 deletions doc/files/package.json.md
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ Ideally you should pick one that is
[OSI](https://opensource.org/licenses/alphabetical) approved.

If your package is licensed under multiple common licenses, use an [SPDX license
expression syntax version 2.0 string](https://npmjs.com/package/spdx), like this:
expression syntax version 2.0 string](https://www.npmjs.com/package/spdx), like this:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems fine.


{ "license" : "(ISC OR GPL-3.0)" }

Expand Down Expand Up @@ -608,7 +608,7 @@ Trying to install another plugin with a conflicting requirement will cause an
error. For this reason, make sure your plugin requirement is as broad as
possible, and not to lock it down to specific patch versions.

Assuming the host complies with [semver](http://semver.org/), only changes in
Assuming the host complies with [semver](https://semver.org/), only changes in
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is ok.

the host package's major version will break your plugin. Thus, if you've worked
with every 1.x version of the host package, use `"^1.0"` or `"1.x"` to express
this. If you depend on features introduced in 1.5.2, use `">= 1.5.2 < 2"`.
Expand Down
4 changes: 2 additions & 2 deletions doc/misc/npm-disputes.md
Original file line number Diff line number Diff line change
Expand Up @@ -102,8 +102,8 @@ here to help.**

If you think another npm publisher is infringing your trademark, such as by
using a confusingly similar package name, email <abuse@npmjs.com> with a link to
the package or user account on [https://npmjs.com](https://npmjs.com). Attach a
copy of your trademark registration certificate.
the package or user account on [https://www.npmjs.com/](https://www.npmjs.com/).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Kinda weird but it's fine.

Attach a copy of your trademark registration certificate.

If we see that the package's publisher is intentionally misleading others by
misusing your registered mark without permission, we will transfer the package
Expand Down
2 changes: 1 addition & 1 deletion doc/misc/npm-registry.md
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,7 @@ effectively implement the entire CouchDB API anyway.

## Is there a website or something to see package docs and such?

Yes, head over to <https://npmjs.com/>
Yes, head over to <https://www.npmjs.com/>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is ok.


## SEE ALSO

Expand Down
2 changes: 1 addition & 1 deletion html/index.html
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@
</head>
<h1>npm</h1>

<p>npm is a package manager for <a href="http://nodejs.org/">node</a>. You can use it to install
<p>npm is a package manager for <a href="https://nodejs.org/">node</a>. You can use it to install
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also ok.

and publish your node programs. It manages dependencies and does other cool stuff.</p>

<h2>Easy Zero Line Install</h2>
Expand Down
2 changes: 1 addition & 1 deletion scripts/install.sh
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
# shell living at /bin/sh.
#
# See this helpful document on writing portable shell scripts:
# http://www.gnu.org/s/hello/manual/autoconf/Portable-Shell.html
# https://www.gnu.org/s/hello/manual/autoconf/Portable-Shell.html
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also ok.

#
# The only shell it won't ever work on is cmd.exe.

Expand Down