[PR #659] [MERGED] New Script: SilverBullet #2827

Closed
opened 2026-02-26 13:31:30 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/community-scripts/ProxmoxVE/pull/659
Author: @dsiebel
Created: 12/3/2024
Status: Merged
Merged: 12/20/2024
Merged by: @MickLesk

Base: mainHead: silverbullet


📝 Commits (9)

  • 62139d9 new script: silverbullet
  • d717694 update copyright and author information
  • 0540760 address code review comments and suggestions
  • 575c246 fix: overwrite existing binary on unzip
  • dd04e15 address remaining review comments
  • 68164f8 add documenation link
  • 2e9538d update header and footer to new standard
  • a9664bb little fixes
  • d38a495 little fixes

📊 Changes

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

View changed files

ct/silverbullet.sh (+61 -0)
install/silverbullet-install.sh (+59 -0)
json/silverbullet.json (+34 -0)

📄 Description

Description

I was looking into Silverbullet today and while installing it in a debian LXC I created this install and update script. Since Silverbullet was already requested (#127) and it's already on the backlog, I figured: why not.

Fixes #127

Type of change

Please check the relevant option(s):

  • Bug fix (non-breaking change that resolves an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (a fix or feature that would cause existing functionality to change unexpectedly)
  • New script (a fully functional and thoroughly tested script or set of scripts.)

Prerequisites

The following efforts must be made for the PR to be considered. Please check when completed:

  • Self-review performed (I have reviewed my code, ensuring it follows established patterns and conventions)
  • Testing performed (I have tested my changes, ensuring everything works as expected)
  • Documentation updated (I have updated any relevant documentation)

Additional Information (optional)

I intentionally decided against using the silverbullet upgrade command, since this would require installing and running the entire deno build. Using pre-built binary just feels better for production use.

Ref: https://silverbullet.md/Install/Deno#Upgrading

If there are other pull requests or discussions related to this change, please link them here:

  • Related PR #

🔄 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/community-scripts/ProxmoxVE/pull/659 **Author:** [@dsiebel](https://github.com/dsiebel) **Created:** 12/3/2024 **Status:** ✅ Merged **Merged:** 12/20/2024 **Merged by:** [@MickLesk](https://github.com/MickLesk) **Base:** `main` ← **Head:** `silverbullet` --- ### 📝 Commits (9) - [`62139d9`](https://github.com/community-scripts/ProxmoxVE/commit/62139d987d7de93110bdec4f9f7c85695c24c52b) new script: silverbullet - [`d717694`](https://github.com/community-scripts/ProxmoxVE/commit/d717694746d1c8457613c0f5656b4279a1edb6bb) update copyright and author information - [`0540760`](https://github.com/community-scripts/ProxmoxVE/commit/05407606b7814d7ddb540125f7a27f98c7cc46ed) address code review comments and suggestions - [`575c246`](https://github.com/community-scripts/ProxmoxVE/commit/575c24625ab9614f58e4f06d2564088b4ff41e53) fix: overwrite existing binary on unzip - [`dd04e15`](https://github.com/community-scripts/ProxmoxVE/commit/dd04e15e6056bb25399257ad34122f863afa154e) address remaining review comments - [`68164f8`](https://github.com/community-scripts/ProxmoxVE/commit/68164f859a7fbee47077094fd94356e6c8703b12) add documenation link - [`2e9538d`](https://github.com/community-scripts/ProxmoxVE/commit/2e9538d66030e8eed8d066a22d215754aa5d0079) update header and footer to new standard - [`a9664bb`](https://github.com/community-scripts/ProxmoxVE/commit/a9664bb94cdcb07c15186f483576020bfe971c53) little fixes - [`d38a495`](https://github.com/community-scripts/ProxmoxVE/commit/d38a49532c12182d99235d2f1e5cda73c46581c6) little fixes ### 📊 Changes **3 files changed** (+154 additions, -0 deletions) <details> <summary>View changed files</summary> ➕ `ct/silverbullet.sh` (+61 -0) ➕ `install/silverbullet-install.sh` (+59 -0) ➕ `json/silverbullet.json` (+34 -0) </details> ### 📄 Description ## Description I was looking into Silverbullet today and while installing it in a debian LXC I created this install and update script. Since Silverbullet was already requested (#127) and it's already on the backlog, I figured: why not. Fixes #127 ## Type of change Please check the relevant option(s): - [ ] Bug fix (non-breaking change that resolves an issue) - [ ] New feature (non-breaking change that adds functionality) - [ ] Breaking change (a fix or feature that would cause existing functionality to change unexpectedly) - [x] New script (a fully functional and thoroughly tested script or set of scripts.) ## Prerequisites The following efforts must be made for the PR to be considered. Please check when completed: - [x] Self-review performed (I have reviewed my code, ensuring it follows established patterns and conventions) - [x] Testing performed (I have tested my changes, ensuring everything works as expected) - [x] Documentation updated (I have updated any relevant documentation) ## Additional Information (optional) I intentionally decided against using the `silverbullet upgrade` command, since this would require installing and running the entire `deno` build. Using pre-built binary just feels better for production use. Ref: https://silverbullet.md/Install/Deno#Upgrading ## Related Pull Requests / Discussions If there are other pull requests or discussions related to this change, please link them here: - Related PR # --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 13:31:30 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/ProxmoxVE#2827
No description provided.