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

Brazil bank calendar #238

Merged
merged 6 commits into from Jul 27, 2017

Conversation

Projects
None yet
2 participants
@fvlima
Copy link
Contributor

commented Jul 25, 2017

Created a brazilian calendar to consider working days for bank transactions. Given a day, this calendar allow to find the next working day by ignoring the weekends, fixed and non fixed holidays and the last working day of the year in Brazil (used only for internal bank transactions). If the given day is a working day, it will be returned.

@brunobord

This comment has been minimized.

Copy link
Contributor

commented Jul 26, 2017

Looks good to me, I didn't know there was such calendars, that's great!
are you okay with your branch? can I merge it?

@fvlima

This comment has been minimized.

Copy link
Contributor Author

commented Jul 26, 2017

Here we have it like a business calendar, considered only to realize bank transactions. Yeah, my branch is ok. :)

@fvlima

This comment has been minimized.

Copy link
Contributor Author

commented Jul 26, 2017

It's my pleasure to contribute with this amazing lib.

@brunobord brunobord merged commit 36427bf into peopledoc:master Jul 27, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.