Add DevJoke by Hauser#876
Conversation
Add DevJoke about C#
📝 WalkthroughWalkthroughAdded a DevJoke block to the end of Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@README.md`:
- Around line 1501-1508: The new text joke block ("#DevJoke" and the lines "Why
do Java programmers wear glasses? Because they don't C#!") was appended to the
bottom but must be placed at the top section: move this joke so it appears
immediately after the README's opening joke separator '***' (i.e., insert the
"#DevJoke" header and its two joke lines right after the top '***' separator),
removing the duplicate at the file end so images remain in the bottom section.
- Around line 1503-1504: The new two-line joke ("Why do Java programmers wear
glasses? Because they don't C#!") is a near-duplicate of the existing joke ("Why
do Java developers wear glasses? Answer: Because they can't C# !"); remove or
replace the added lines with a genuinely different joke instead of the same
punchline. Locate the duplicate snippet in the README (match the exact text of
the new lines) and swap it for a fresh joke that does not reuse "C#" as the
punchline, verifying the replacement is unique compared to the existing "can't
C#" variant.
- Line 1501: The markdown token "#DevJoke" is malformed and inconsistent with
the rest of the file; change it to either remove the per-joke top-level heading
or convert it to a proper heading with a space and appropriate level (e.g., "#
DevJoke" or "## DevJoke") to match the file's existing structure—locate the
exact token "#DevJoke" and replace it accordingly to restore proper Markdown
rendering and consistency.
| #DevJoke | ||
|
|
||
| Why do Java programmers wear glasses? | ||
| Because they don't C#! | ||
|
|
||
| *** | ||
|
|
||
|
|
There was a problem hiding this comment.
Joke added at the bottom — contribution guidelines require text jokes at the top.
Line 5 of this README explicitly states:
"Add text jokes at the TOP of the page, add your joke followed by
***."
The new content is appended at the very end of the file. Text jokes should be inserted near the top (after the *** on line 7), not at the bottom (which is reserved for images per the same instructions).
🧰 Tools
🪛 LanguageTool
[style] ~1504-~1504: Using many exclamation marks might seem excessive (in this case: 47 exclamation marks for a text that’s 20125 characters long)
Context: ...mers wear glasses? Because they don't C#! ***
(EN_EXCESSIVE_EXCLAMATION)
🪛 markdownlint-cli2 (0.21.0)
[warning] 1501-1501: No space after hash on atx style heading
(MD018, no-missing-space-atx)
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@README.md` around lines 1501 - 1508, The new text joke block ("#DevJoke" and
the lines "Why do Java programmers wear glasses? Because they don't C#!") was
appended to the bottom but must be placed at the top section: move this joke so
it appears immediately after the README's opening joke separator '***' (i.e.,
insert the "#DevJoke" header and its two joke lines right after the top '***'
separator), removing the duplicate at the file end so images remain in the
bottom section.
| Software and cathedrals are much the same — first we build them, then we pray. | ||
| *** | ||
|
|
||
| #DevJoke |
There was a problem hiding this comment.
Malformed Markdown heading — missing space after #.
#DevJoke will not render as a heading in Markdown; it requires a space: # DevJoke. Additionally, this per-joke heading is inconsistent with the rest of the file, where no individual joke uses its own #-level heading, and would render as a competing top-level <h1> alongside the file's existing # Dev Joke title.
✏️ Proposed fix
-#DevJoke
-
-Why do Java programmers wear glasses?
-Because they don't C#!
-
-***
+Why do Java programmers wear glasses?
+Because they don't C#!
+
+***📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| #DevJoke | |
| Why do Java programmers wear glasses? | |
| Because they don't C#! | |
| *** |
🧰 Tools
🪛 markdownlint-cli2 (0.21.0)
[warning] 1501-1501: No space after hash on atx style heading
(MD018, no-missing-space-atx)
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@README.md` at line 1501, The markdown token "#DevJoke" is malformed and
inconsistent with the rest of the file; change it to either remove the per-joke
top-level heading or convert it to a proper heading with a space and appropriate
level (e.g., "# DevJoke" or "## DevJoke") to match the file's existing
structure—locate the exact token "#DevJoke" and replace it accordingly to
restore proper Markdown rendering and consistency.
| Why do Java programmers wear glasses? | ||
| Because they don't C#! |
There was a problem hiding this comment.
Duplicate joke — this already exists in the file.
An identical joke appears at Lines 553–555:
Question - Why do Java developers wear glasses?
Answer: Because they can't C# !
The new addition is effectively the same punchline with only minor wording differences ("programmers" vs. "developers", "don't" vs. "can't"). Please replace this with a genuinely new joke to match the PR's stated intent.
🧰 Tools
🪛 LanguageTool
[style] ~1504-~1504: Using many exclamation marks might seem excessive (in this case: 47 exclamation marks for a text that’s 20125 characters long)
Context: ...mers wear glasses? Because they don't C#! ***
(EN_EXCESSIVE_EXCLAMATION)
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@README.md` around lines 1503 - 1504, The new two-line joke ("Why do Java
programmers wear glasses? Because they don't C#!") is a near-duplicate of the
existing joke ("Why do Java developers wear glasses? Answer: Because they can't
C# !"); remove or replace the added lines with a genuinely different joke
instead of the same punchline. Locate the duplicate snippet in the README (match
the exact text of the new lines) and swap it for a fresh joke that does not
reuse "C#" as the punchline, verifying the replacement is unique compared to the
existing "can't C#" variant.
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@README.md`:
- Around line 1-8: Remove the accidental scratch content at the top of the
README by deleting the stray header "# LHD Init" and the list items "1. You" and
"2. Me" so the file begins directly with the intended project title "# Dev Joke"
(i.e., remove the three lines introduced as personal test notes).
---
Duplicate comments:
In `@README.md`:
- Around line 1511-1512: The new joke "Why do Java programmers wear glasses?
Because they don't C#!" is a near-duplicate of the existing line "Why do Java
developers wear glasses? Because they can't C# !"; replace the new entry with a
genuinely original joke (modify the string that currently reads "Why do Java
programmers wear glasses? Because they don't C#!" to a different, unique one) so
the README no longer contains duplicated punchlines.
- Line 1509: The line containing the malformed heading "#DevJoke" should be
fixed: either remove the leading "#" so it becomes a normal sentence "DevJoke"
to match the rest of the jokes, or if you intended a subheading add a space and
make it consistent (e.g., "## Dev Joke"); update the "#DevJoke" token
accordingly to ensure proper Markdown rendering and consistency with the
top-level "Dev Joke" title.
- Around line 1509-1516: Move the new joke block headed "#DevJoke" from the
bottom to directly after the first '***' separator near the top of README.md (so
the text joke appears at the top of the page as required), ensuring the joke
content remains unchanged; also remove the duplicate block at the bottom and
confirm image sections remain after all text-joke blocks at the file end.
| # LHD Init | ||
|
|
||
| 1. You | ||
| 2. Me | ||
|
|
||
|
|
||
|
|
||
|
|
There was a problem hiding this comment.
Remove accidental scratch content prepended to the README.
Lines 1–8 introduce # LHD Init, 1. You, and 2. Me at the very top of the file. This content has no relation to the DevJokes project, creates a duplicate <h1> before the existing # Dev Joke title, and will be visible to all repository visitors. These lines appear to be personal test notes committed by mistake and should be removed entirely.
🐛 Proposed fix
-# LHD Init
-
-1. You
-2. Me
-
-
-
-
# Dev Joke📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| # LHD Init | |
| 1. You | |
| 2. Me | |
| # Dev Joke |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@README.md` around lines 1 - 8, Remove the accidental scratch content at the
top of the README by deleting the stray header "# LHD Init" and the list items
"1. You" and "2. Me" so the file begins directly with the intended project title
"# Dev Joke" (i.e., remove the three lines introduced as personal test notes).
1 participant
Add DevJoke about C#
Summary by CodeRabbit