sourencho / ungroup_game

A multiplayer game about temporary alliances written with a custom engine in C++ and SFML
13 stars 1 forks source link

Add terminal UI to server #193

Closed copacetic closed 4 years ago

copacetic commented 4 years ago

Description

Description of changes

Fixes

Fixes #X

sourencho commented 4 years ago

Before you merge this, i'll run it through clang-format on my computer to fix the style

sourencho commented 4 years ago

According to travis the smoke test failed. I think it's because it relies on stdout to print something specific

1.16s$ bash -c 'if ! xvfb-run --server-args="-screen 0 1280x1024x24" -e /dev/stdout -a node tests/integration/smoke_tests.js; then >&2 echo "Fail"; exit 1; fi'
eoke test failed!
Fail
The command "bash -c 'if ! xvfb-run --server-args="-screen 0 1280x1024x24" -e /dev/stdout -a node tests/integration/smoke_tests.js; then >&2 echo "Fail"; exit 1; fi'" exited with 1.
Done. Your build exited with 1.
sourencho commented 4 years ago

I'll merge #191 after you merge this into master

copacetic commented 4 years ago

🥳

On Sun, Apr 26, 2020, 11:55 AM Souren Papazian notifications@github.com wrote:

Merged #193 https://github.com/SourenP/ungroup_game/pull/193 into master.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/SourenP/ungroup_game/pull/193#event-3273257611, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAIBOZ4OMAP2K5HPKXNW2M3ROR7S3ANCNFSM4MQUPKBQ .