Skip to content
This repository has been archived by the owner on May 14, 2024. It is now read-only.

Update SogeBot #2233

Merged
merged 1 commit into from
Apr 23, 2023
Merged

Update SogeBot #2233

merged 1 commit into from
Apr 23, 2023

Conversation

QuintenQVD0
Copy link
Collaborator

Description

  • change install and server image to nodejs 18
  • make the install script downgrade npm to v8 (mirror of there docker file)
  • change the build command (mirror of the docker file)

closes #2231

Checklist for all submissions

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Have you tested and reviewed your changes with confidence that everything works?
  • Did you branch your changes and PR from that branch and not from your master branch?
    • If not, why?:
  • You verify that the start command applied does not use a shell script
    • If some script is needed then it is part of a current yolk or a PR to add one
  • The egg was exported from the panel

Copy link
Collaborator

@parkervcp parkervcp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@parkervcp parkervcp merged commit 3e0a1d9 into master Apr 23, 2023
@parkervcp parkervcp deleted the SogeBot branch April 23, 2023 17:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Install Bug]: Sogebot
2 participants