Skip to content
This repository was archived by the owner on May 25, 2021. It is now read-only.

Conversation

@ajacksified
Copy link
Contributor

Update to latest inrupt logo.

@ajacksified ajacksified requested review from NSeydoux and Vinnl June 26, 2020 00:46
Copy link
Contributor

@Vinnl Vinnl left a comment

Choose a reason for hiding this comment

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

LGTM, might want to check your npm version and fix that if you used a different one than was specified though.

"abbrev": {
"version": "1.1.1",
"resolved": false,
"resolved": "",
Copy link
Contributor

@Vinnl Vinnl Jun 26, 2020

Choose a reason for hiding this comment

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

Do you have nvm (or equivalent for your OS) set up to automatically switch to the right version? Because these changes look like they're caused by inconsistent npm versions. If that's because someone else used a different version than specified in .nvmrc then that's fine, but if it's because you used a different version, this will cause such noise and potentially conflicts in future PRs :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, I'm on the node version on the .nvmrc with this commit - but likely wasn't last time I updated package-lock.json. 😬

@ajacksified ajacksified merged commit 0387777 into master Jun 26, 2020
@ajacksified ajacksified deleted the chore/update-inrupt-logo branch June 26, 2020 13:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants