inbuxa/inbuxa-admin
1
0 Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 22 Wiki Activity Sponsor

build: correct the Dockerfile's note on dist/ #101

Merged
jcoffey-dev merged 1 commits from docker/dist-comment into main 2026-10-10 18:59:44 +00:00
Conversation 0 Commits 1 Files Changed 1
+4 -3
jcoffey-dev commented 2026-10-10 18:53:37 +00:00
Owner
Copy link
Copy source

Comment only. The Dockerfile said dist/ is committed "for the convenience of people serving the tree directly". It isn't (.gitignore keeps it out), so the line reads as if a checkout can be served as-is when it can't. The reason for building inside the image stays.

Comment only. The Dockerfile said `dist/` is committed "for the convenience of people serving the tree directly". It isn't (`.gitignore` keeps it out), so the line reads as if a checkout can be served as-is when it can't. The reason for building inside the image stays.
jcoffey-dev added 1 commit 2026-10-10 18:53:37 +00:00
build: correct the Dockerfile's note on dist/
ci / build (pull_request) Skipped
Details
ci / publish (pull_request) Skipped
Details
ci / notices (pull_request) Successful in 57s
Details
github/ci (branch) GitHub Actions
Details
ci / github (pull_request) Successful in 1m35s
Details
9d693fd135 
It said dist/ is committed for people serving the tree directly. It isn't: .gitignore keeps it out, so there is no built tree to serve. The reason for building in the image stands.
jcoffey-dev merged commit 8da7ae1080 into main 2026-10-10 18:59:44 +00:00
jcoffey-dev deleted branch docker/dist-comment 2026-10-10 18:59:44 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something isn't working
 dependencies
Dependency updates
 documentation
Docs, README or site copy
 duplicate
Already tracked elsewhere
 enhancement
New feature or improvement
 good first issue
Good for newcomers
 help wanted
Contributions welcome
 invalid
Not a valid issue
 needs info
Waiting on the reporter for details
 question
Needs an answer, not a code change
 security
Security issue or hardening
 translation
New language or translation fixes
 wontfix
Will not be worked on
priority
p1
High priority
priority
p2
Medium priority
priority
p3
Low priority
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
jcoffey-dev (John Coffey)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: inbuxa/inbuxa-admin#101