Discuss AU holidays about definitions HOT 2 OPEN

Ignore the above, just read #35 and the attached code and now I better understand. In ANZAC Day's case we are adding additional dates for particular regions, whereas for NYD we want to remove a date from applying for a particular region.

Looking at Adam's original commit, how about this as an API:

  - name: New Year's Day
    regions: [au, au_nsw, au_vic, au_act, au_sa, au_wa, au_nt, au_qld, au_tas]
    mday: 1
    observed: au_nyd_observed_to_monday_if_weekend(date, region)
    function: au_nyd_function_to_monday_if_weekend(date, region)

I should learn more about the holidays internals - I don't really know how feasible this is - but my idea is that we'd pass an optional region argument to all observed/function methods. You can then use this to return nil if you want to. So in this case, au_nyd_observed_to_monday_if_weekend would return nil for :au_tas, and au_nyd_function_to_monday_if_weekend would return nil for anything except :au_tas. They would probably all call to_monday_if_weekend internally for all other cases.

Most importantly (to me) this wouldn't require upstream code changes, unlike the :exact suggestion

Thoughts?

from definitions.

You say that any :au holiday will also apply to :au_tas too. How does that work with ANZAC Day? According to my link there's an :au wide definition, then more specific ones for the states that handle how it is observed.

(I haven't been following this closely over the last two weeks so forgive me if this has already been answered.)

from definitions.