Skip to content

Use system temp directory for temporary font downloads instead of $HOME#69

Draft
Copilot wants to merge 2 commits into
mainfrom
copilot/use-temp-directory-for-font-downloads
Draft

Use system temp directory for temporary font downloads instead of $HOME#69
Copilot wants to merge 2 commits into
mainfrom
copilot/use-temp-directory-for-font-downloads

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 17, 2026

Temporary download directories created by Install-NerdFont use $HOME as the base path. If the command fails before cleanup runs, these directories persist in the user's home folder where no OS-level cleanup reclaims them.

  • Switch base path from $HOME to [System.IO.Path]::GetTempPath() — cross-platform .NET API resolving to $Env:TEMP (Windows) / /tmp (Linux/macOS)
# Before
$tempPath = Join-Path -Path $HOME -ChildPath "NerdFonts-$guid"

# After
$tempPath = Join-Path -Path ([System.IO.Path]::GetTempPath()) -ChildPath "NerdFonts-$guid"

GUID naming and clean block logic unchanged.

Copilot AI changed the title [WIP] Use system temp directory for font downloads instead of $HOME Use system temp directory for temporary font downloads instead of $HOME May 17, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Super-linter summary

Language Validation result
CHECKOV Pass ✅
GITHUB_ACTIONS Pass ✅
GITLEAKS Pass ✅
GIT_MERGE_CONFLICT_MARKERS Pass ✅
JSON Pass ✅
MARKDOWN Pass ✅
NATURAL_LANGUAGE Pass ✅
POWERSHELL Pass ✅
PRE_COMMIT Pass ✅
SPELL_CODESPELL Pass ✅
TRIVY Pass ✅
YAML Pass ✅

All files and directories linted successfully

For more information, see the GitHub Actions workflow run

Powered by Super-linter

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.

Use system temp directory for temporary font downloads instead of $HOME

2 participants