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

Fixed A2S_INFO request format for Source protocol. #578

Closed
wants to merge 6 commits into from

Conversation

@scrutinizer-notifier
Copy link

The inspection completed: No new issues

@coveralls
Copy link

coveralls commented May 25, 2021

Coverage Status

Coverage remained the same at 87.798% when pulling a6438fd on Ruslan-700:v3 into af6641c on Austinb:v3.

@stale
Copy link

stale bot commented Jun 23, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the Stale Marked as stale label Jun 23, 2021
@Austinb Austinb added Bug This request does fix a bug. v3 and removed Stale Marked as stale labels Jun 26, 2021
@Austinb Austinb self-assigned this Jun 26, 2021
@bumbummen99
Copy link
Collaborator

Any updates on this?

@Austinb
Copy link
Owner

Austinb commented Feb 11, 2022

All tests must pass before I will merge. Its possible the data set for Crysis 2 needs to be fixed due to your change .

@bumbummen99
Copy link
Collaborator

bumbummen99 commented Feb 11, 2022

All tests must pass before I will merge. Its possible the data set for Crysis 2 needs to be fixed due to your change .

Looking to the history test data for Crysis 2 never was added or if it was where can i find it?
bumbummen99@178f142#diff-30160c0d679d8a7c0b6df392c982ea903e71dda407037d3c33e84386b282d3fb

Fix: bumbummen99@4a47319

@Austinb
Copy link
Owner

Austinb commented Feb 18, 2022

I believe this is fixed in #591

@Austinb Austinb closed this Feb 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug This request does fix a bug. v3
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants