Skip to content

Some Goonstation improvements(?)#1

Closed
pali6 wants to merge 15 commits into
BeeStation:masterfrom
goonstation:master
Closed

Some Goonstation improvements(?)#1
pali6 wants to merge 15 commits into
BeeStation:masterfrom
goonstation:master

Conversation

@pali6

@pali6 pali6 commented May 18, 2023

Copy link
Copy Markdown

Based on discussion in the OpenDream Discord here the changes from our fork. Accompanied by Crossedfall/crossed-cogs#23

  • Support for arbitrary versions.
  • Normalization of indentation so you can use e.g. two spaces.
  • Cleanup of the output.
  • Multiple Docker paths are tried on Linux

Let me know if any of this sucks or you don't want it and I'll remove it from the PR.

Comment thread compile.sh
Comment thread listener.py Outdated
Comment thread listener.py
@pali6

pali6 commented May 18, 2023

Copy link
Copy Markdown
Author

I was bypassing Dockerfile because just passing the relevant arguments to run was the first solution I found back when I was doing this a couple of years ago. Now it should be done more correctly (I hope) by building the beestation/byond image and then building the test image from the Dockerfile.

@goonstation goonstation closed this by deleting the head repository Feb 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants