[PR #2398] [MERGED] fix dockerfile creating non oss builds #20105

Closed
opened 2026-05-21 21:30:00 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/fosrl/pangolin/pull/2398
Author: @Lokowitz
Created: 2/2/2026
Status: Merged
Merged: 2/5/2026
Merged by: @oschwartz10612

Base: devHead: fix-dockerfile


📝 Commits (1)

📊 Changes

5 files changed (+18 additions, -21 deletions)

View changed files

📝 .github/workflows/linting.yml (+1 -1)
📝 .github/workflows/test.yml (+2 -8)
📝 Dockerfile (+2 -2)
📝 esbuild.mjs (+7 -1)
📝 package.json (+6 -9)

📄 Description

Community Contribution License Agreement

By creating this pull request, I grant the project maintainers an unlimited,
perpetual license to use, modify, and redistribute these contributions under any terms they
choose, including both the AGPLv3 and the Fossorial Commercial license terms. I
represent that I have the right to grant this license for all contributed content.

Description

There is a bug in building non oss build because i missed the -b tag in esbuild.
Tried to use the server/build.ts const but import is not working so i asked ai.
I am sure you know a better way to extract the build value from there.

If you want i revert this and add 3 npm run commands instead.

I am currently testing Docker Hardened Image. Needs some more refactoring, are you interested?
https://github.com/Lokowitz/pangolin/pull/551


🔄 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/fosrl/pangolin/pull/2398 **Author:** [@Lokowitz](https://github.com/Lokowitz) **Created:** 2/2/2026 **Status:** ✅ Merged **Merged:** 2/5/2026 **Merged by:** [@oschwartz10612](https://github.com/oschwartz10612) **Base:** `dev` ← **Head:** `fix-dockerfile` --- ### 📝 Commits (1) - [`b92f5d3`](https://github.com/fosrl/pangolin/commit/b92f5d3c91b9ec87f3e13e51f06272b92f215b1e) fix dockerfile ### 📊 Changes **5 files changed** (+18 additions, -21 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/linting.yml` (+1 -1) 📝 `.github/workflows/test.yml` (+2 -8) 📝 `Dockerfile` (+2 -2) 📝 `esbuild.mjs` (+7 -1) 📝 `package.json` (+6 -9) </details> ### 📄 Description ## Community Contribution License Agreement By creating this pull request, I grant the project maintainers an unlimited, perpetual license to use, modify, and redistribute these contributions under any terms they choose, including both the AGPLv3 and the Fossorial Commercial license terms. I represent that I have the right to grant this license for all contributed content. ## Description There is a bug in building non oss build because i missed the -b tag in esbuild. Tried to use the server/build.ts const but import is not working so i asked ai. I am sure you know a better way to extract the build value from there. If you want i revert this and add 3 npm run commands instead. I am currently testing Docker Hardened Image. Needs some more refactoring, are you interested? https://github.com/Lokowitz/pangolin/pull/551 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-05-21 21:30:00 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/pangolin#20105