[PR #480] [MERGED] fix: create path to config dir if does not exist #493

Closed
opened 2026-02-16 02:15:13 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/run-llama/llama_deploy/pull/480
Author: @masci
Created: 3/7/2025
Status: Merged
Merged: 3/7/2025
Merged by: @masci

Base: mainHead: massi/fix-config-path


📝 Commits (1)

  • c5f7715 fix: create path to config dir if does not exist

📊 Changes

2 files changed (+10 additions, -0 deletions)

View changed files

📝 llama_deploy/cli/internal/config.py (+2 -0)
📝 tests/cli/internal/test_config.py (+8 -0)

📄 Description

The config path returned by platformdirs might not exist (the most likely scenario in docker containers), make sure we create the full path to the default config file when the CLI runs for the first time.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/run-llama/llama_deploy/pull/480 **Author:** [@masci](https://github.com/masci) **Created:** 3/7/2025 **Status:** ✅ Merged **Merged:** 3/7/2025 **Merged by:** [@masci](https://github.com/masci) **Base:** `main` ← **Head:** `massi/fix-config-path` --- ### 📝 Commits (1) - [`c5f7715`](https://github.com/run-llama/llama_deploy/commit/c5f771593ec69b768aecd2c4ab961f619b6c0494) fix: create path to config dir if does not exist ### 📊 Changes **2 files changed** (+10 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `llama_deploy/cli/internal/config.py` (+2 -0) 📝 `tests/cli/internal/test_config.py` (+8 -0) </details> ### 📄 Description The config path returned by `platformdirs` might not exist (the most likely scenario in docker containers), make sure we create the full path to the default config file when the CLI runs for the first time. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-16 02:15:13 -05:00
yindo closed this issue 2026-02-16 02:15:13 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: run-llama/llama_deploy#493