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

Supprime le fichier des jours fériés et le script de génération #1222

Merged
merged 2 commits into from
Nov 28, 2018

Conversation

Morendil
Copy link
Contributor

  • Changement mineur.
  • Zones impactées : assets, cotisations_sociales.
  • Détails :
    • Supprime le fichier holidays.py qui contenait une liste de jours fériés français
    • Supprime le script de génération de ce fichier
    • Supprime l'usage, jamais testé, de ces jours fériés dans coefficient_proratisation

  • ce script et ses données étaient utilisés exclusivement pour coefficient_proratisation, variable au calcul déjà fort complexe
  • l'incidence des jours férié sur ces calculs n'était ni spécifiée ni testée, or les règles sous-jacentes sont assez complexes: le 1er mai, le lundi de Pentecôte ont un statut spécial; l'ancienneté du salarié entre en ligne de compte, etc.
  • il vaudrait mieux dépendre plus explicitement de workalendar au lieu de chercher à précalculer les jours fériés dans un script qu'on aurait certainement oublié de faire tourner pour mettre à jour les dates dans holidays.py (et l'échéance était assez proche, juin 2019)

@Morendil Morendil force-pushed the fix-holidays branch 2 times, most recently from 776f49a to f3af171 Compare November 25, 2018 15:54
@Morendil Morendil merged commit 522dfe3 into master Nov 28, 2018
@Morendil Morendil deleted the fix-holidays branch November 28, 2018 07:26
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