Open bergnerjonas opened 2 months ago
[!IMPORTANT]
Review skipped
Auto reviews are disabled on base/target branches other than the default branch.
Please check the settings in the CodeRabbit UI or the
.coderabbit.yaml
file in this repository. To trigger a single review, invoke the@coderabbitai review
command.You can disable this status message by setting the
reviews.review_status
tofalse
in the CodeRabbit configuration file.
The changes involve a modification in the get_data
function within the src/pystatis/table.py
file. The logic for determining the date format has been updated to consider both the language and the database name. Specifically, the German date format ("%d.%m.%Y"
) will only be applied if the database name is not "zensus"
; otherwise, the format defaults to "%Y-%m-%d"
when the database is "zensus"
. This adjustment affects how date strings are handled based on these parameters.
Files | Change Summary |
---|---|
src/pystatis/table.py |
Modified the get_data function to change the logic for the date_format parameter based on the combination of language and db_name . |
In the land of data bright,
A change was made to set things right.
For German dates, a twist we see,
In "zensus," new formats set us free!
Hooray for logic, clear and grand,
A hop, a skip, we take a stand! 🐇✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 81.30%. Comparing base (
1dcd903
) to head (cb10d9f
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@bergnerjonas what is the current status of this?
This is a quick fix for the date parser in zensus, which uses YYYY-MM-DD regardless of language settings.
Summary by CodeRabbit
New Features
Bug Fixes