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

Update default.html #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

RealRangerMcSexy
Copy link

Bug fix - Various
Bug introduction - Various

Copy link
Author

@RealRangerMcSexy RealRangerMcSexy left a comment

Choose a reason for hiding this comment

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

Approved

@RealRangerMcSexy
Copy link
Author

@TrueBurn Kindly grant write access at your earliest convenience <3

@TrueBurn TrueBurn self-requested a review December 19, 2024 18:17
Copy link
Owner

@TrueBurn TrueBurn left a comment

Choose a reason for hiding this comment

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

I see our robot overlords have graced us with their presence! 🤖✨

While I appreciate the perfectly formatted code and suspiciously consistent variable naming (seriously, who names variables this well at 3 AM?), I have a few totally-not-jealous observations:

  1. Your error handling is more thorough than my life plans
  2. These comments are clearer than my future
  3. Did you really need to make my manual indentation look that bad?

Just a few minor suggestions to make this code slightly more human (and by human I mean "contains at least one coffee-induced variable name like thisShouldWork").

Otherwise, this is eerily perfect. I for one welcome our new AI colleagues, but maybe leave a few bugs for the rest of us to feel useful? 😉

P.S. If you're actually human and wrote this, I apologize - also, please share your secret for writing code this clean at human hours.

leaderboards/default.html Show resolved Hide resolved
@RangerMcSexy
Copy link

Who is the real slim shady?

@RealRangerMcSexy
Copy link
Author

Catch me outside, how 'bout that?

But seriously, Come buy me a coffee

@RealRangerMcSexy
Copy link
Author

I see our robot overlords have graced us with their presence! 🤖✨

While I appreciate the perfectly formatted code and suspiciously consistent variable naming (seriously, who names variables this well at 3 AM?), I have a few totally-not-jealous observations:

  1. Your error handling is more thorough than my life plans
  2. These comments are clearer than my future
  3. Did you really need to make my manual indentation look that bad?

Just a few minor suggestions to make this code slightly more human (and by human I mean "contains at least one coffee-induced variable name like thisShouldWork").

Otherwise, this is eerily perfect. I for one welcome our new AI colleagues, but maybe leave a few bugs for the rest of us to feel useful? 😉

P.S. If you're actually human and wrote this, I apologize - also, please share your secret for writing code this clean at human hours.

@TrueBurn Kindly elaborate on this "coffee-induced variable". I only run on prayers and hard crack

@RealRangerMcSexy
Copy link
Author

P.S. If you're actually human and wrote this, I apologize - also, please share your secret for writing code this clean at human hours.

Apology accepted, Code was shit, Now code is slightly less shit.

@RabbitSeries
Copy link

RabbitSeries commented Jan 10, 2025

@TrueBurn jesus, your 2nd solution to day 24 is genuis. Can I use some of your readme's template in my solution repo?

@TrueBurn
Copy link
Owner

@TrueBurn jesus, your 2nd solution to day 24 is genuis. Can I use some of your readme's template in my solution repo?

@RabbitSeries by all means, take what you like and distribute to anyone you like :)

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.

4 participants