code review

  • 66
    Taking future proofing to the next level
  • 8
  • 8
    πŸ€”his not wrong.... but you and the next guy looking after that won't be around to see it.
  • 19
    @C0D4 after human extinction, when some Alien come down to earth and take a look at this commit he probably think humans are useless.
  • 26
    and now - the real code review:
    Why, The, *FUCK*, Do, You, Need, A, Const, For, Number, Of, Days, In, A, Month?

    this smells like a reinvention of the wheel. but in a new and exiting square shape this time.
  • 29
    31 is wrong anyway.

    The right value is 30.4375. And it's just as useless as 31.
  • 27
    You must be using the APIs provided by our solar system for getting the days in a month

    Also, if you are planning for long term, look at the v3 API docs of the Milky way... That will be safe even after 5 bn years

    Long term thinking you see
  • 4
  • 0
    @magicMirror bcz just a number doesn't tell what it is hence const to improve readability.
  • 1
    just hook up a solar system simulation
  • 2
    There's no npm package for that yet?
  • 0
    @xsid I missed your comment.
    I will say this again:
    Why, Do, You, Need, The, Fuken, Days, Number, Const, Var, Int, String, Or, Any, Other, Form?
  • 2
    @magicMirror did it occurred to you that you are a csv guy
  • 0
    @xsid what? the extra use of commas? well.... maybe.
    I prefer newline delmited cursing though.
Add Comment