Is the compute time of recalculating it over and over again more or less burdensome than receiving multiple PRs and spending the time to address each of them β including closing five (or more next year!) of them?
If you answer that I think you'll have a good reason to either merge or close the Time.current.year PR π€
I actually quite enjoy this odd throwback ritual, but acknowledging that in pure software/productivity terms it should be computed. Merged a PR that does that.
Since we're eyeing a future where people can stand up their own version of our open source platform community, hardcoded dates will eventually be bad legacy code.
Since we're hoping that future happens at some point in 2019, makes sense to go computed now. π
For further actions, you may consider blocking this person and/or reporting abuse
We're a place where coders share, stay up-to-date and grow their careers.
I'd definitely see the hardcoded date as legacy code, if not now, someday. May as well merge the
Time.current.year
PR.Is the compute time of recalculating it over and over again more or less burdensome than receiving multiple PRs and spending the time to address each of them β including closing five (or more next year!) of them?
If you answer that I think you'll have a good reason to either merge or close the
Time.current.year
PR π€Abhaha I agree with you, let's keep it hard-coded or put it dynamic but in a partial, cached until 2019 ends π€
I actually quite enjoy this odd throwback ritual, but acknowledging that in pure software/productivity terms it should be computed. Merged a PR that does that.
Since we're eyeing a future where people can stand up their own version of our open source platform community, hardcoded dates will eventually be bad legacy code.
Since we're hoping that future happens at some point in 2019, makes sense to go computed now. π