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

v8.2.0 #4115

Merged
merged 17 commits into from
Dec 2, 2021
Merged

v8.2.0 #4115

merged 17 commits into from
Dec 2, 2021

Conversation

wraithgar
Copy link
Member

@wraithgar wraithgar commented Dec 2, 2021

v8.2.0 (2021-12-02)

Features

Bug Fixes

Documentation

Dependencies

yuqu and others added 17 commits November 22, 2021 06:58
PR-URL: #4081
Credit: @idleberg
Close: #4081
Reviewed-by: @wraithgar
  * fix: always load localPrefix

PR-URL: #4102
Credit: @wraithgar
Close: #4102
Reviewed-by: @lukekarrys
PR-URL: #4078
Credit: @wraithgar
Close: #4078
Reviewed-by: @lukekarrys
This decouples the log file writing from the terminal logging. We now
open an append only file at the start of the process and stream logs to
it. We still only display the log file message in timing mode or if
there is an error, but the file is still written regardless.

All logging now goes through `proc-log` and this is the first step to
removing `npmlog`. For now `npmlog` is still used for the terminal
logging but with a shim in front of it to make it easier to test and
use in conjunction with `proc-log`. Ref: npm/statusboard#366

This also refactors many of the tests to always use an explicit
`t.testdir` for their cache since the file is opened on each `new Npm()`.
Tests are also refactored to use more of `MockNpm` with behavior to
add config items and load `npm` if necessary. A new fixture `mockGlobals`
was also added to make much of this more ergonomic. Ref: npm/statusboard#410

Closes npm/statusboard#411
Closes npm/statusboard#367

PR-URL: #4062
Credit: @lukekarrys
Close: #4062
Reviewed-by: @wraithgar
PR-URL: #4109
Credit: @wraithgar
Close: #4109
Reviewed-by: @lukekarrys
PR-URL: #4113
Credit: @darcyclarke
Close: #4113
Reviewed-by: @lukekarrys
@wraithgar wraithgar requested a review from a team as a code owner December 2, 2021 21:43
@wraithgar wraithgar merged commit 4dbeb00 into latest Dec 2, 2021
@nlf nlf deleted the release/v8.2.0 branch March 28, 2022 20:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants