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

feat(info): show location for Web Cache #26205

Merged
merged 9 commits into from
Nov 20, 2024
Merged

Conversation

haturatu
Copy link
Contributor

@haturatu haturatu commented Oct 13, 2024

Hi. team.
I have confirmed that there is another cache directory. However, this is not visible when running deno info. Therefore, I sent this request, but if you have any better suggestions or if this is unnecessary, please let me know.

Closes #26181

Checked cargo test -vv

alleycat:[hrtaro]:~/deno$ ./target/debug/deno info
DENO_DIR location: /home/hrtaro/.cache/deno
Remote modules cache: /home/hrtaro/.cache/deno/remote
npm modules cache: /home/hrtaro/.cache/deno/npm
Emitted modules cache: /home/hrtaro/.cache/deno/gen
Language server registries cache: /home/hrtaro/.cache/deno/registries
Origin storage: /home/hrtaro/.cache/deno/location_data
Deno Cache: /tmp/deno_cache

@CLAassistant
Copy link

CLAassistant commented Oct 13, 2024

CLA assistant check
All committers have signed the CLA.

@haturatu
Copy link
Contributor Author

It's not smart, so I'll fix it again this week.

@dsherret dsherret added this to the 2.1.0 milestone Oct 21, 2024
@bartlomieju bartlomieju requested review from bartlomieju and removed request for dsherret November 19, 2024 15:25
@bartlomieju bartlomieju self-assigned this Nov 19, 2024
@bartlomieju bartlomieju changed the title docs(console): deno info add Deno Cache: /tmp/deno_cache feat(info): show location for Web Cache Nov 19, 2024
Copy link
Member

@bartlomieju bartlomieju left a comment

Choose a reason for hiding this comment

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

I applied some changes, thanks!

@bartlomieju bartlomieju enabled auto-merge (squash) November 19, 2024 23:20
cli/tools/info.rs Outdated Show resolved Hide resolved
@bartlomieju bartlomieju enabled auto-merge (squash) November 20, 2024 00:18
@bartlomieju bartlomieju merged commit 429f392 into denoland:main Nov 20, 2024
17 checks passed
@haturatu
Copy link
Contributor Author

I’m sorry I couldn’t get to this sooner because I’ve been busy with moving and other things.
Thank you for handling it!🙌

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.

/tmp/deno_cache does not exist in deno info.
4 participants