Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding Full Mandatory Mexican holidays #20

Merged
merged 2 commits into from
Jan 19, 2024
Merged

Conversation

dantes4ur
Copy link
Contributor

@dantes4ur dantes4ur commented Jan 17, 2024

Nation-wide holidays for Mexico from the official government publication:

https://www.gob.mx/profedet/es/articulos/sabes-cuales-son-los-dias-de-descanso-obligatorio-para-este-2024

Article 74 of the Federal Labor Law indicates the following mandatory rest days:

I. January 1st;

II. The first Monday of February in commemoration of February 5th;

III. The third Monday of March in commemoration of March 21st;

IV. May 1st;

V. September 16th;

VI. The third Monday of November in commemoration of November 20th;

VII. October 1st every six years, when it corresponds to the transmission of the Federal Executive Power;

VIII. December 25th, and

IX. The day determined by federal and local electoral laws, in the case of ordinary elections, to conduct the electoral day.

Aside from these, any additional days are considered optional.

When dynamic holidays are incorporated into the package, we will be able to enhance and update this pull request accordingly.

@dantes4ur dantes4ur changed the title Adding Mexican holidays Adding Full Mandatory Mexican holidays Jan 17, 2024
Copy link
Member

@Nielsvanpach Nielsvanpach left a comment

Choose a reason for hiding this comment

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

Thanks! Could you rebase this branch, in order to run the tests here?

As the Mexican public holidays seem a bit more difficult, I'll leave this PR open until some people upvoted this as approval.

return 'mx';
}

/** @return array<string, CarbonImmutable> */
Copy link
Member

Choose a reason for hiding this comment

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

this can be removed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The comments right?

For that part, done

Copy link

@andresgaviria99 andresgaviria99 left a comment

Choose a reason for hiding this comment

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

LGTM

@Nielsvanpach
Copy link
Member

Thanks! Could you solve the merge conflicts?

@dantes4ur
Copy link
Contributor Author

Thanks! Could you solve the merge conflicts?

Already fixed, some other branch was merged with optional days.

@Nielsvanpach
Copy link
Member

Great, thank you!

@Nielsvanpach Nielsvanpach merged commit edc8d39 into spatie:main Jan 19, 2024
7 of 8 checks passed
@Nielsvanpach
Copy link
Member

I made some adjustments, could you verify if the governmentChangeDate() change is correct?
https://github.com/spatie/holidays/pull/140/files

@dantes4ur
Copy link
Contributor Author

I made some adjustments, could you verify if the governmentChangeDate() change is correct? https://github.com/spatie/holidays/pull/140/files

LGTM!

ssmusoke pushed a commit to ssmusoke/holidays that referenced this pull request Jan 20, 2024
* feat(Mexico.php): rebase main to put mandatory days

* feat(Mexico.php): finished dates
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants