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

➕ Added calender for India. #724

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jaydeepkarale
Copy link

Currently only major holidays which are observed across India are added. Regional and variable holidays based on Hindu Calender to be added in future

refs #

For information, read and make sure you're okay with the Contributing guidelines.

  • [ X] Tests with a significant number of years to be tested for your calendar.
  • [ X] Docstrings for the Calendar class and specific methods.
  • [ X] Use the workalendar.registry_tools.iso_register decorator to register your new calendar using ISO codes (optional).
  • [ X] Calendar country / label added to the README.md file.
  • [ X] Changelog amended with a mention like: "Added <country> by @pseudo (#)". Note Please do NOT change the version number here. It's the project maintainers' duty.
  • Tests with a significant number of years to be tested for your calendar.
  • Changelog amended with a mention describing your changes. Note Please do NOT change the version number here. It's the project maintainers' duty.

…bserved across India are added. Regional and variable holidays based on Hindu Calender to be added in future
Copy link
Member

@brunobord brunobord left a comment

Choose a reason for hiding this comment

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

only a slight issue with you change in the .gitignore file.
also, you may want to rebase from latest master, which has dropped compatibility with Python 3.6, and has added compatibility with Python 3.10 + 3.11.

If you need any help about rebasing, please tell me, I'll take your PR over.

Sorry for the very long delay, I've (again) neglected this project for too long.

@@ -11,3 +11,4 @@ pyproject.toml
.cache/
.pytest-cache/
.pytest_cache/
.idea/
Copy link
Member

Choose a reason for hiding this comment

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

this has nothing to do with this contribution.
could you please remove this change?

@rehanguha
Copy link

Is this an active PR? I can consider the changes and raise the same PR.

@jaydeepkarale
Copy link
Author

Hey @rehanguha happy for you to take over this

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.

None yet

3 participants