Skip to content

Commit

Permalink
fix: remove "hashAlgorithm" from flatOptions
Browse files Browse the repository at this point in the history
BREAKING CHANGE: the hard-coded "hashAlgorithm" value is no longer being
passed through flatOptions

Nothing in npm is using this.
  • Loading branch information
wraithgar authored and lukekarrys committed Jul 26, 2023
1 parent ece52a3 commit db91a77
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 5 deletions.
3 changes: 0 additions & 3 deletions workspaces/config/lib/definitions/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -25,9 +25,6 @@ const flatten = (obj, flat = {}) => {
: /* istanbul ignore next - not configurable property */ undefined
flat.nodeBin = process.env.NODE || process.execPath

// XXX should this be sha512? is it even relevant?
flat.hashAlgorithm = 'sha1'

return flat
}

Expand Down
2 changes: 0 additions & 2 deletions workspaces/config/test/definitions/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,6 @@ t.test('flatten', t => {
'//foo.bar.com:_authToken': 'foobarbazquuxasdf',
npmBin: '/path/to/npm',
nodeBin: '/path/to/node',
hashAlgorithm: 'sha1',
})

mockGlobals(t, {
Expand All @@ -52,7 +51,6 @@ t.test('flatten', t => {
'//foo.bar.com:_authToken': 'foobarbazquuxasdf',
npmBin: '/path/to/npm',
nodeBin: '/usr/local/bin/node.exe',
hashAlgorithm: 'sha1',
})

t.end()
Expand Down

0 comments on commit db91a77

Please sign in to comment.