Make dynamic days more readable

[pil0u]

Summary of changes and context

Title.

Sanity checks

• Linters pass
• Tests pass
• Related GitHub issues are linked in the description

[Aquaj]

En vrai je suis pas fan. Ça ne marche que parce que le begin_time tombe sur le 1er du mois.
Je sais que l'AoC ne tombera jamais un 2 décembre ou quoi, mais du coup pour moi le code n'exprime pas bien la réalité là, il aurait aussi fallu changer le nommage de begin_time ou quelque chose du genre.

[pil0u]

@Aquaj Je ne suis pas 100% fan non plus, c'est quoi le "mieux" selon toi ? Une méthode genre

# aoc.rb

def self.release_time(day)
  Time.new(2023, 12, day, 0, 0, 0, "-05:00")
end

qu'on call ensuite

Aoc.release_time(day)
# ou
Aoc.release_time(day) + 48.hours

Ça a plus de gueule en effet 😁

[Aquaj]

J'avais pas forcément d'alternative à part rollback la PR, ce que tu proposes est plus propre déjà (car ça permet de centraliser toute la logique de manipulation des jours au même endroit: aoc.rb).
Bonne idée 👍

[pil0u]

#372