• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

npm / cli / #1420
97%
latest: 97%

Build:
Build:
LAST BUILD BRANCH: nlf/arborist-validation
DEFAULT BRANCH: latest
Ran 04 Mar 2021 09:00PM UTC
Jobs 1
Files 117
Run time 24s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

pending completion
#1420

push

ruyadorno
Remove unused arguments on various function calls

I checked cli's code with Typescript using the tsconfig below. The
compiler found a few arguments that are not used, so I removed them. In
the case of `npm whoami`, it is clearer that it ignores its `args`
and instead relies on `npm.flatOptions`.

```json
{
    "compilerOptions": {
        "moduleResolution": "node",
        "module": "commonjs",
        "resolveJsonModule": true,
        "target": "es2019",
        "noImplicitAny": false,
        "noImplicitThis": true,
        "strict": true,
        "maxNodeModuleJsDepth": 0,
        "noEmit": true,
        "allowJs": true,
        "checkJs": true,
        "types": ["node"],
        "lib": ["esnext"]
    },
    "include": ["lib"]
}
```

PR-URL: https://github.com/npm/cli/pull/2766
Credit: @sandersn
Close: #2766
Reviewed-by: @nlf, @ruyadorno, @Matausi29

2883 of 2960 branches covered (97.4%)

Branch coverage included in aggregate %.

5003 of 5108 relevant lines covered (97.94%)

20.44 hits per line

Jobs
ID Job ID Ran Files Coverage
1 #1420.1 04 Mar 2021 09:00PM UTC 0
97.74
Source Files on build #1420
Detailed source file information is not available for this build.
  • Back to Repo
  • b33c760c on github
  • Prev Build on release/v7.6.1
  • Next Build on release/v7.6.1
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2025 Coveralls, Inc