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

Rendu labo David Jaquet et Vincent Guidoux #88

Open
wants to merge 102 commits into
base: master
Choose a base branch
from

Conversation

Nortalle
Copy link

@Nortalle Nortalle commented Apr 8, 2018

No description provided.

romain_silvestri and others added 30 commits March 25, 2018 11:08
…contrib

Adding 4 new tests for V2 protocol
wasadigi and others added 28 commits April 5, 2018 06:48
6 Tests for the V2 Protocol added by Frueh and Muaremi
automated tests to specify the behaviour of the v2 protocol
Ajout tests V2 Protocol - Mathieu Jee, Lionel Burgbacher
Protocol V2 Tests - Adrien Allemand, Loyse Krug
…trib

PR tests v2 from Guillaume Blanco and Patrick Neto
@Nortalle Nortalle changed the title On regarde Rendu labo David Jaquet et Vincent Guidoux Apr 8, 2018
@Nortalle
Copy link
Author

Nortalle commented Apr 9, 2018

Il manque à la ligne 156 :

socket = null;

dans la fonction closeConnection() de la classe RouletteV1ClientImpl

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.