Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve missing gameserver warning #743

Merged
merged 4 commits into from
Jul 4, 2023
Merged

Conversation

XAMPPRocky
Copy link
Collaborator

Outputting the debug info of the whole servers prints a lot of noise, this reduces it to just what info we use for the endpoint.

@XAMPPRocky XAMPPRocky force-pushed the ep/improve-missing-warning branch from 7373200 to f282ad2 Compare May 29, 2023 14:06
@XAMPPRocky XAMPPRocky enabled auto-merge (squash) July 4, 2023 11:37
@github-actions github-actions bot added size/xs and removed size/m labels Jul 4, 2023
@quilkin-bot
Copy link
Collaborator

Build Succeeded 🥳

Build Id: 09b7f2cb-17a6-4ee9-a5d8-53dc1e5d4677

The following development images have been built, and will exist for the next 30 days:

To build this version:

git fetch [email protected]:googleforgames/quilkin.git pull/743/head:pr_743 && git checkout pr_743
cargo build

@XAMPPRocky XAMPPRocky merged commit 18b9bda into main Jul 4, 2023
@markmandel markmandel deleted the ep/improve-missing-warning branch July 5, 2023 17:52
@markmandel markmandel added area/user-experience Pertaining to developers trying to use Quilkin, e.g. cli interface, configuration, etc kind/cleanup Refactoring code, fixing up documentation, etc labels Jul 7, 2023
swermin pushed a commit to swermin/quilkin that referenced this pull request Oct 16, 2024
* Remove shutdown_rx from downstream loop

* Improve missing gameserver warning

* Add name as property in warning
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/user-experience Pertaining to developers trying to use Quilkin, e.g. cli interface, configuration, etc kind/cleanup Refactoring code, fixing up documentation, etc size/xs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants