Queen Elizabeth II passing Bank Holiday about holidays

[Jonic]

@hlascelles Thanks for the speedy update!

Does this still count as having been fixed without an updated release of this Gem? Without your PR merging to definitions, and a new release of the Gem, technically this issue hasn't been resolved yet, but closing this issue might give the impression that it has been. Should this issue stay open until a new release has been published?

In the meantime I've taken the additional config you've added to manually provide a definition in our app, but hopefully there'll be a new release of the Gem before then.

[ttwo32]

@hlascelles Thank you for letting me know that.:)
If you don't mind, could you please submit a PR at holidays/definitions?

[justinjones]

@ttwo32 I've updated the definitions for the AU version of this holiday: holidays/definitions#232

[hlascelles]

Added... holidays/definitions#235

Also, a fix for the KE holidays: holidays/definitions#236

[hlascelles]

Good point, will leave open!

[WorkSutton]

@hlascelles Looks like this can be closed now as the PR has been merged in the definitions file and this commit ed17774 pulled it altogether.

Only thing I cannot see is a new tagged/released version has not been pushed even though the version has changed.

[ppeble]

It has been pushed, the version is v8.6.0 in rubygems. I will update this releases page, sorry about it! Please try pulling it and let us know if it works for you!

[WorkSutton]

@ppeble Yes I was able to pull the gem using bundle update holidays --conservative and it update from 8.5.0 to 8.6.0.

I just expected to see a tag in releases/tags, but maybe I was confused on that.

Thanks for getting this merged in and published today. Very much appreciated 👍

[ppeble]

No, you aren't wrong, I had to add it just now. That's why you couldn't see it, I forgot! Thanks for reminding me. 🙇

[hlascelles]

Thanks for the gem release! 👍