Cleanup old enums #982
Cleanup old enums #982
Conversation
fpagnoux
commented
May 15, 2018
fpagnoux
commented
- Changement mineur
- Détails :
- Supprime les énums utilisées en syntaxe pré-V4, qui n'est plus supportée.
| @@ -28,7 +28,7 @@ Chaque évolution sera documentée par les élements suivants : | |||
| - `Correction d'un crash` : Impacte tous les réutilisateurs. | |||
| - `Changement mineur` : Refactoring, métadonnées… N'a aucun impact sur les réutilisateurs. | |||
|
|
|||
| - Dans le cas d'une `Évolution du système socio-fiscal`, il est ensuite précisé : | |||
| - **Dans le cas d'une `Évolution du système socio-fiscal`** , il est ensuite précisé : | |||
There was a problem hiding this comment.
@maukoquiroga attention, la "zone impactée" n'a de sens que pour une évolution de la législation. Ça aide rapidement les réutilisateurs à voir si un changement les intéresse ou non. Le but n'est pas de tracker les fichiers.
En général, il est aussi préférable de faire valider le Changelog lors de la PR, et de ne pas l'écrire après coup, c'est en fait sans doute un des morceaux de diffs les plus importants d'une PR :).
fpagnoux
force-pushed
the
cleanup-old-enums
branch
from
9db029c to
fef6b8f
Compare
Oui, c'est sûr que les clés sont peu expressives tel quel. Je ne connais pas trop l'usage des cerfa fields, donc je suis resté sur les clés entières, mais on peut sans doute trouver mieux. |
|
En gros dans ce cas c'est vous, conjoint, 1ere personne à charge, 2nde personne à charge etc. |
