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.

Sign up for GitHub

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

Move reconnect handling to faf server accessor #3061

Merged
merged 2 commits into from
Nov 27, 2023

Fix commons version

7ca7086
Select commit
Loading
Failed to load commit list.
Merged

Move reconnect handling to faf server accessor #3061

Fix commons version
7ca7086
Select commit
Loading
Failed to load commit list.
This check has been archived and is scheduled for deletion. Learn more about checks retention
Codecov / codecov/patch succeeded Nov 27, 2023 in 0s

77.58% of diff hit (target 58.86%)

View this Pull Request on Codecov

77.58% of diff hit (target 58.86%)

Annotations

Check warning on line 102 in src/main/java/com/faforever/client/remote/FafServerAccessor.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/com/faforever/client/remote/FafServerAccessor.java#L102 

Added line #L102 was not covered by tests

Check warning on line 120 in src/main/java/com/faforever/client/remote/FafServerAccessor.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/com/faforever/client/remote/FafServerAccessor.java#L119-L120 

Added lines #L119 - L120 were not covered by tests

Check warning on line 156 in src/main/java/com/faforever/client/remote/FafServerAccessor.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/com/faforever/client/remote/FafServerAccessor.java#L155-L156 

Added lines #L155 - L156 were not covered by tests

Check warning on line 158 in src/main/java/com/faforever/client/remote/FafServerAccessor.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/com/faforever/client/remote/FafServerAccessor.java#L158 

Added line #L158 was not covered by tests

Check warning on line 194 in src/main/java/com/faforever/client/remote/FafServerAccessor.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/com/faforever/client/remote/FafServerAccessor.java#L190-L194 

Added lines #L190 - L194 were not covered by tests
View more details on Codecov