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

Feature/Fix XTKS 2021 holidays according to JPX calendar #5

Merged
merged 1 commit into from
Dec 15, 2020

Conversation

sercant
Copy link

@sercant sercant commented Dec 4, 2020

Updated according to JPX 2021 Market Holidays.

Ref: https://www.jpx.co.jp/english/corporate/about-jpx/calendar/

@sercant sercant requested a review from yoshiso December 4, 2020 10:37
@sercant
Copy link
Author

sercant commented Dec 4, 2020

oh need to sync master with upstream

@sercant sercant changed the base branch from master to feature/sync-with-upstream December 4, 2020 10:41
@sercant sercant force-pushed the feature/fix-jpx-2021-calendar branch from b72fa85 to 2dcb33a Compare December 4, 2020 10:48
@sercant sercant changed the base branch from feature/sync-with-upstream to master December 4, 2020 10:49
@sercant sercant changed the base branch from master to feature/sync-with-upstream December 4, 2020 10:49
@sercant sercant requested a review from saubhik December 4, 2020 10:49
Base automatically changed from feature/sync-with-upstream to master December 9, 2020 03:51
@sercant sercant force-pushed the feature/fix-jpx-2021-calendar branch from 2dcb33a to 078d5b4 Compare December 9, 2020 03:55
Copy link

@saubhik saubhik left a comment

Choose a reason for hiding this comment

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

LGTM!

@sercant sercant self-assigned this Dec 15, 2020
@sercant sercant merged commit 19c46c5 into master Dec 15, 2020
@sercant sercant deleted the feature/fix-jpx-2021-calendar branch November 11, 2021 02:19
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.

2 participants