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

Rename fiscal calendar options #58

Merged
merged 2 commits into from
Sep 28, 2023
Merged

Rename fiscal calendar options #58

merged 2 commits into from
Sep 28, 2023

Conversation

royshoo
Copy link
Contributor

@royshoo royshoo commented Sep 28, 2023

start_month -> fiscal_year_start_month
first_day_of_week -> first_day_of_fiscal_week
and remove the parameter which was ignored, and will be rejected with the new release

`start_month` -> `fiscal_year_start_month`
`first_day_of_week` -> `first_day_of_fiscal_week`
and remove the parameter which was ignored, and will be rejected with
the new release
Co-authored-by: Adrian Vogelsgesang <[email protected]>
@royshoo royshoo merged commit 51836c9 into upcoming Sep 28, 2023
@royshoo royshoo deleted the xshu-fiscal-rename branch September 28, 2023 20:17
wolfroediger pushed a commit that referenced this pull request Oct 9, 2023
* Rename fiscal calendar options

`start_month` -> `fiscal_year_start_month`
`first_day_of_week` -> `first_day_of_fiscal_week`
and remove the parameter which was ignored, and will be rejected with
the new release

* Update website/docs/releases.md

Co-authored-by: Adrian Vogelsgesang <[email protected]>

---------

Co-authored-by: Adrian Vogelsgesang <[email protected]>
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.

3 participants