Skip to content

Conversation

wraithgar
Copy link
Member

Found a bug refactoring the tests.

libnpmaccess mutates the response from the server, and the completion
code was not looking for the right value.

@wraithgar wraithgar requested a review from a team as a code owner April 14, 2022 16:00
@wraithgar wraithgar force-pushed the gar/undeprecate-tests branch from 2c4b5fd to cde6a3a Compare April 14, 2022 16:01
@wraithgar wraithgar changed the title fix: undeprecate completion Apr 14, 2022
Found a bug refactoring the tests.

libnpmaccess mutates the response from the server, and the completion
code was not looking for the right value.
@wraithgar wraithgar force-pushed the gar/undeprecate-tests branch from cde6a3a to cd596ce Compare April 14, 2022 16:02
Copy link
Contributor

@nlf nlf left a comment

Choose a reason for hiding this comment

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

the title threw me for a loop for a second asking myself "why are we deprecating completion?" before i realized what it actually meant lol

this looks good! good catch!

@ljharb
Copy link
Contributor

ljharb commented Apr 14, 2022

ha, same

@lukekarrys lukekarrys merged commit b10462e into latest Apr 14, 2022
@lukekarrys lukekarrys deleted the gar/undeprecate-tests branch April 14, 2022 22:30
@ruyadorno ruyadorno mentioned this pull request Apr 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants