Skip to content

Conversation

@PPsyrius
Copy link
Collaborator

@PPsyrius PPsyrius commented Nov 5, 2024

Proposed change

  • Add in all possible exact holiday entries for Brunei Darussalam for years 1998-2025 (excluding 2004-2005 due to missing information).
  • Remove non-observed 3rd day of Eid al-Fitr holiday entries for pre-2012 (an extra date was added in 2012 via ( 8 ) JPM/SK/2012 before being officially added in all entries from 2013 onwards).

Note

These are the list of missing "Surat Keliling Jabatan Perdana Menteri" from Prime Minister Office of Brunei Darussalam's Online Archive (usually referred to as ( [BILLNO] ) JPM/SK/[YEAR] ), likely lost media altogether.

  • JPMSK11-2003 "Hari-Hari Kelapasan Awam tahun 2004" (Public Holidays in 2004): simply missing from the archives at the moment.
  • JPMSK11-2004 "Hari-Hari Kelapasan Awam tahun 2005" (Public Holidays in 2005): the current link is a misplaced Public Holidays announcement for 2006 instead.
  • JPMSK(10?-12?)-2008 "Hari-Hari Kelapasan Awam tahun 2009" (Public Holidays in 2009): this is thankfully more recent than other missing entries so a school holidays calendar copy exists, though having the original government source would still be appreciated here.

Also, since the accuracy of https://www.timeanddate.com/holidays/brunei/ Islamic holidays entries pre-2023 are now proved to be questionable at best, I would like to remove the prefilled data for 2004 and 2005 for now to restore the (estimated) suffix for those years if possible. 👀

Type of change

  • New country/market holidays support (thank you!)
  • Supported country/market holidays update (calendar discrepancy fix, localization)
  • Existing code/documentation/test/process quality improvement (best practice, cleanup, refactoring, optimization)
  • Dependency update (version deprecation/pin/upgrade)
  • Bugfix (non-breaking change which fixes an issue)
  • Breaking change (a code change causing existing functionality to break)
  • New feature (new holidays functionality in general)

Checklist

@codecov
Copy link

codecov bot commented Nov 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (60966ea) to head (66f6a5a).
Report is 4 commits behind head on dev.

Additional details and impacted files
@@            Coverage Diff            @@
##               dev     #2107   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          192       192           
  Lines        11577     11586    +9     
  Branches      1741      1746    +5     
=========================================
+ Hits         11577     11586    +9     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Collaborator

@KJhellico KJhellico left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's great research! I have no suggestions, just a few thoughts :)

@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 6, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
4.0% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

Copy link
Collaborator

@KJhellico KJhellico left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. 👍

@arkid15r arkid15r added this pull request to the merge queue Nov 7, 2024
Copy link
Collaborator

@arkid15r arkid15r left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Merged via the queue into vacanza:dev with commit 93f8441 Nov 7, 2024
28 of 29 checks passed
@PPsyrius PPsyrius deleted the bn_holidays_2025 branch November 7, 2024 02:51
@KJhellico KJhellico mentioned this pull request Nov 18, 2024
mstuttgart pushed a commit to multidadosti-erp/python-holidays that referenced this pull request Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants