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

docs: update npm-cache directory location #38373

Merged
merged 2 commits into from
Jul 26, 2023
Merged

docs: update npm-cache directory location #38373

merged 2 commits into from
Jul 26, 2023

Conversation

jouni
Copy link
Contributor

@jouni jouni commented Jul 25, 2023

The location of the npm-cache/_libvips directory seems to be incorrect in the documentation. On my Windows 11 setup, it’s located under AppData/Local, not AppData/Roaming:

Screenshot 2023-07-25 at 15 28 42

Whereas the AppData/Roaming directory doesn't have have an npm-cache directory at all:

Screenshot 2023-07-25 at 15 28 59

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Jul 25, 2023
@pieh
Copy link
Contributor

pieh commented Jul 25, 2023

I'm not on Windows right now, but googling npm cache location it seems depending on some factors (like used npm cli toggles) it can end up in either place, so maybe both should be mentioned?

@pieh pieh added type: documentation An issue or pull request for improving or updating Gatsby's documentation and removed status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer labels Jul 25, 2023
@jouni
Copy link
Contributor Author

jouni commented Jul 25, 2023

Okay, thanks for checking that! Yeah, I suppose both could be mentioned.

Copy link
Contributor

@pieh pieh left a comment

Choose a reason for hiding this comment

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

Thanks!

@pieh pieh merged commit fd8de34 into gatsbyjs:master Jul 26, 2023
31 of 32 checks passed
@jouni jouni deleted the patch-1 branch July 26, 2023 09:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: documentation An issue or pull request for improving or updating Gatsby's documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants