[PR #1139] [MERGED] add shebangs to migration and server scripts #32616

Closed
opened 2026-06-15 10:08:46 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/fosrl/pangolin/pull/1139
Author: @SigmaSquadron
Created: 7/27/2025
Status: Merged
Merged: 7/27/2025
Merged by: @oschwartz10612

Base: mainHead: push-xlmpuutwtnuy


📝 Commits (1)

  • 9e87c42 add shebangs to migration and server scripts

📊 Changes

3 files changed (+3 additions, -0 deletions)

View changed files

📝 server/index.ts (+1 -0)
📝 server/setup/migrationsPg.ts (+1 -0)
📝 server/setup/migrationsSqlite.ts (+1 -0)

📄 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

In NixOS, we wrap these files in a bash script to allow users to just run them as normal executables, instead of calling them as arguments to Node.JS. In our build scripts, we just add the shebang after the files have been compiled, but adding it upstream will allow all Pangolin users to just run ./server.mjs to start their Pangolin instances.

How to test?

Build Pangolin, and check if all *.mjs files have shebangs. (and not just cli.mjs)


🔄 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/1139 **Author:** [@SigmaSquadron](https://github.com/SigmaSquadron) **Created:** 7/27/2025 **Status:** ✅ Merged **Merged:** 7/27/2025 **Merged by:** [@oschwartz10612](https://github.com/oschwartz10612) **Base:** `main` ← **Head:** `push-xlmpuutwtnuy` --- ### 📝 Commits (1) - [`9e87c42`](https://github.com/fosrl/pangolin/commit/9e87c42d0cabc241ee13b3769ca75ce3d7a1259f) add shebangs to migration and server scripts ### 📊 Changes **3 files changed** (+3 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `server/index.ts` (+1 -0) 📝 `server/setup/migrationsPg.ts` (+1 -0) 📝 `server/setup/migrationsSqlite.ts` (+1 -0) </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 In NixOS, we wrap these files in a bash script to allow users to just run them as normal executables, instead of calling them as arguments to Node.JS. In our build scripts, we just [add the shebang](https://github.com/NixOS/nixpkgs/blob/d299df66b63ba4e6f2f98f5b1e13bedd7b748d8b/pkgs/by-name/fo/fosrl-pangolin/package.nix#L100) after the files have been compiled, but adding it upstream will allow all Pangolin users to just run ./server.mjs to start their Pangolin instances. ## How to test? Build Pangolin, and check if all `*.mjs` files have shebangs. (and not just `cli.mjs`) --- <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-06-15 10:08:46 -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#32616