tteck / Proxmox

Proxmox VE Helper-Scripts
https://Helper-Scripts.com
MIT License
11.68k stars 1.81k forks source link

Add new LXC: Matterbridge | Add new LXC: MeTube #3177

Closed MickLesk closed 2 weeks ago

MickLesk commented 2 weeks ago

I wanted to make you aware that I am meticulous when it comes to merging code into the main branch, so please don't take it personally if I reject your request.

Description

New Matterbridge LXC added

Port: 8283 Source: https://github.com/Luligu/matterbridge/

A short code review would be nice. So far everything works, I have removed unnecessary packages and updated NodeJS to the latest dearmor gpg version.

Request: https://github.com/tteck/Proxmox/discussions/50#discussioncomment-9743382

image

Fixes # (issue)

Type of change

Please delete options that are not relevant.


I wanted to make you aware that I am meticulous when it comes to merging code into the main branch, so please don't take it personally if I reject your request.

Description

New MeTube LXC added

Special note:

Port: 8081 Disk space: 10 GB by default (otherwise it fills up too quickly with YouTube downloads :D) Source: https://github.com/alexta69/metube

A short code review would be nice. So far everything works, I have removed unnecessary packages and updated NodeJS to the latest dearmor gpg version.

Request: https://github.com/tteck/Proxmox/discussions/50#discussioncomment-9745632

image

Fixes # (issue)

Type of change

Please delete options that are not relevant.

MickLesk commented 2 weeks ago

@tteck, sorry my github desktop fucked up and do some shit, i need to recreate the pull request https://github.com/tteck/Proxmox/pull/3170

tteck commented 2 weeks ago

NOPE. Each has to be its own Pull Request.