$ your roast results

// here's what we think of your code

shame score

4/10

1 critical issues

2 warnings

3 good practices

// submitted code

Language: javascript

code.js

1

2

3

4

5

if (lastIncreasingSequence.length > longestFound.length) {
  return lastIncreasingSequence
} else {
  return longestFound
}

// detailed_analysis

Oh look, someone discovered the `Enter` key and decided to use it as a weapon against readability. This is five whole lines of code to perform a comparison that any developer with a functioning prefrontal cortex and a basic grasp of a ternary operator would have finished in one. Are you being paid by the line, or are you just trying to hit a word count to satisfy some deep-seated trauma from high school English? It’s like watching someone build a Rube Goldberg machine just to flip a light switch—it’s technically functional, but I lose a little bit of my will to live just looking at it. Seriously, what’s next? Are you going to write a 400-page manual on how to blink? The `else` block here is particularly offensive; it’s a monument to your inability to write concise, modern JavaScript. We have the `?` and `:` characters on our keyboards for a reason, but I guess you prefer the "toddler’s first coding project" aesthetic. This logic is so basic it probably thinks black pepper is too spicy. Use `return lastIncreasingSequence.length > longestFound.length ? lastIncreasingSequence : longestFound;` and stop wasting the CPU's time—and mine. SEVERITY: 4/10 (It works, but it’s an aesthetic hate crime.)