Skip to content

fix(public_dir): respect public_dir setting when loading VM#1802

Merged
flexiondotorg merged 1 commit into
masterfrom
public_dir
Jan 23, 2026
Merged

fix(public_dir): respect public_dir setting when loading VM#1802
flexiondotorg merged 1 commit into
masterfrom
public_dir

Conversation

@flexiondotorg

Copy link
Copy Markdown
Member
  • Read public_dir from the sourced VM config into PUBLIC
  • Preserve existing PUBLIC when public_dir is not set
  • Prevent PUBLIC being empty when a VM specifies public_dir

Fixes #1634

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • I have performed a self-review of my code

- Read public_dir from the sourced VM config into PUBLIC
- Preserve existing PUBLIC when public_dir is not set
- Prevent PUBLIC being empty when a VM specifies public_dir

Fixes #1634

Signed-off-by: Martin Wimpress <martin@wimpress.org>

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 1 file

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

@flexiondotorg flexiondotorg merged commit d83db98 into master Jan 23, 2026
5 checks passed
@flexiondotorg flexiondotorg deleted the public_dir branch January 23, 2026 17:56
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.

bug: public_dir directive in configuration does not seem to have an effect

1 participant