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

Boljše preverjanje datumov pri uvozu MIZŠ podatkov #103

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

Boljše preverjanje datumov pri uvozu MIZŠ podatkov #103

wants to merge 539 commits into from

Conversation

ikolar
Copy link
Collaborator

@ikolar ikolar commented Feb 19, 2021

  • popravki očitno napačnih letnic (20, 2121)
  • opozorilo, če je datum začetka odsotnosti za datumom konca odsotnosti (cca 600 primerov)
  • opozorilo, če sta datum začetka ali konca preveč v prihodnosti (10 oz. 90 dni), kar verjetno pomeni zatipk
  • v generiranem csv-ju ne bo več ure (napaka od zadnjič)

Verjetno hočete izločiti popravek izdelanih csv-jev. Naj to raje naredi vaš bot.

1613568164
=======
1613568229
>>>>>>> d5e22fd290f15082b09a8f100618b4fe493b8a1f
Copy link
Contributor

Choose a reason for hiding this comment

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

tole je ostanek conflict resolvinga... treba fixat

"Start date {} is after the stop date {} in line:\n{}".format(
start_date, stop_date, row
)
)
Copy link
Contributor

Choose a reason for hiding this comment

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

Could be either

logger.warning(f'Start date {start_date} is after the stop date {stop_date} in line:\n{row}')

(shorter)... or ...

logger.warning('Start date %s is after the stop date %s in line:\n%s', start_date, stop_date, row)

to leverage filtering using logging services where you can do aggregations for this message regardless of parameter values.

@jurijbajzelj
Copy link
Contributor

Also, commits can be squashed into one :)

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