27 concurrent connections v2

[safstromo]

Code for starting a new virtual thread for each connection
resolve #27

[MikaelEdwartz]

Code looks good, what do you think of moving the connection functionality to another class for more OOP and readability?

[safstromo]

Code looks good, what do you think of moving the connection functionality to another class for more OOP and readability?

I think its a good idea, we could wait for merging day and create a separate issue for that?
i think most of the clutter will disapear when the commands are sorted and the printlines are removed

[MikaelEdwartz]

Code looks good, what do you think of moving the connection functionality to another class for more OOP and readability?

I think its a good idea, we could wait for merging day and create a separate issue for that? i think most of the clutter will disapear when the commands are sorted and the printlines are removed

Smart! I agree, there's probably going to be an influx of java code soon, let's get that to work first and refactor after

[jLereback]

Looking good! I'll request a change on some formatting and a refactored method 🤗

[jLereback]

Great work! 😄

[cchriss123]

Looks a lot cleaner and better now. Great work!

[kappsegla]

Java code looks good but when looking at the pom.xml file it looks like every line almost has been replaced. Must be indentation that has changed from space to tab or something like that so git diff marks every line as replaced?
I think it might be editorconfig that has a problem and specifies tabs instead of spaces. Made an issue #62 about the problem.

[sonarcloud]

SonarCloud Quality Gate failed.   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
4 Code Smells

0.0% Coverage
0.0% Duplication

[jLereback]

Looks good, the pom.xml has the correct format 👍