Skip to content

Initial version of the Server - #83

Open
adamtornhill wants to merge 1 commit into
masterfrom
build-the-server
Open

Initial version of the Server#83
adamtornhill wants to merge 1 commit into
masterfrom
build-the-server

Conversation

@adamtornhill

Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gates Failed
❌ Enforce advisory code health rules (1 file with Complex Conditional)
❌ New code is healthy (1 new file with code health below 10.00)

Gates Passed
✅ 4 Quality Gates Passed

See analysis details in CodeScene

Reason for failure
Enforce advisory code health rules Violations Code Health Impact
Server.java 1 advisory rule 10.00 → 9.69 Suppress
New code is healthy Violations Code Health Impact
Server.java 1 rule 10.00 → 9.69 Suppress

Quality Gate Profile: Custom Configuration
Want more control? Customize Code Health rules or catch issues early with our IDE extension and CLI tool.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gates Failed
❌ New code is healthy (1 new file with code health below 10.00)
❌ Enforce advisory code health rules (1 file with Complex Conditional)

Gates Passed
✅ 4 Quality Gates Passed

See analysis details in CodeScene

Reason for failure
New code is healthy Violations Code Health Impact
Server.java 1 rule 10.00 → 9.69 Suppress
Enforce advisory code health rules Violations Code Health Impact
Server.java 1 advisory rule 10.00 → 9.69 Suppress

Quality Gate Profile: Custom Configuration
Want more control? Customize Code Health rules or catch issues early with our IDE extension and CLI tool.

Comment thread src/Server/Server.java
}

public void start(ServerConfig c) {
if (c != null && c.isEnabled() && c.canCommunicate()) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

❌ New issue: Complex Conditional
start has 1 complex conditionals with 2 branches, threshold = 2

Suppress

@lissy93

lissy93 commented Feb 18, 2025

Copy link
Copy Markdown

Following along in the tutorial, but the link errors. seems to be on your private staging.
image

@josf

josf commented Sep 12, 2025

Copy link
Copy Markdown

Following along in the tutorial, but the link errors. seems to be on your private staging. image

Hello @lissy93,
I'm sorry we didn't see your comment on this PR earlier, but thanks for pointing out the problem with the link. Do you remember which tutorial you were doing when you landed here? That would allow us to clear up this problem.

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.

3 participants