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

feat: Upgrade date-fns to 2.30.0 #2730

Merged
merged 3 commits into from
Dec 19, 2024
Merged

feat: Upgrade date-fns to 2.30.0 #2730

merged 3 commits into from
Dec 19, 2024

Conversation

Merkur39
Copy link
Member

@Merkur39 Merkur39 commented Dec 18, 2024

See breaking changes here

@Merkur39 Merkur39 force-pushed the feat/upgrade_date-fns branch from 03b37a1 to 120a0a2 Compare December 18, 2024 15:53
Copy link

bundlemon bot commented Dec 18, 2024

BundleMon

Files updated (1)
Status Path Size Limits
transpiled/react/stylesheet.css
16.93KB (+13.83KB +446.31%) -
Unchanged files (2)
Status Path Size Limits
dist/cozy-ui.min.css
20.23KB +10%
dist/cozy-ui.utils.min.css
10.91KB +10%

Total files change +13.83KB +40.41%

Groups updated (1)
Status Path Size Limits
transpiled/react/**
666.04KB (+14.18KB +2.18%) -

Final result: ✅

View report in BundleMon website ➡️


Current branch size history | Target branch size history

@Merkur39 Merkur39 force-pushed the feat/upgrade_date-fns branch 3 times, most recently from ff8f0ab to 71aac3f Compare December 19, 2024 07:52
@Merkur39
Copy link
Member Author

@JF-Cozy J'ai finalement converti la commit en BC, car le format de date change également.
Il ne peut être automatiquement traité comme le type du premier argument.

@Merkur39 Merkur39 force-pushed the feat/upgrade_date-fns branch 3 times, most recently from 0f062f2 to 5733fa9 Compare December 19, 2024 09:37
@JF-Cozy
Copy link
Collaborator

JF-Cozy commented Dec 19, 2024

@JF-Cozy J'ai finalement converti la commit en BC, car le format de date change également. Il ne peut être automatiquement traité comme le type du premier argument.

Les BC côté date-fns ont des titres. Je pense qu'on devrait aider un peu ici et mettre dans le texte de notre BC, les BC auxquels faut faire référence dans la doc date-fns. Tous ne sont pas pertinent.

@Merkur39 Merkur39 force-pushed the feat/upgrade_date-fns branch 4 times, most recently from fd3d955 to 0961233 Compare December 19, 2024 10:44
@Merkur39
Copy link
Member Author

@JF-Cozy J'ai finalement converti la commit en BC, car le format de date change également. Il ne peut être automatiquement traité comme le type du premier argument.

Les BC côté date-fns ont des titres. Je pense qu'on devrait aider un peu ici et mettre dans le texte de notre BC, les BC auxquels faut faire référence dans la doc date-fns. Tous ne sont pas pertinent.

Ok je vais link uniquement la doc de format(en v2.30.0), puisque c'est ce que nous utilisons et le seul changement qui peut impacter les apps

@Merkur39 Merkur39 force-pushed the feat/upgrade_date-fns branch 6 times, most recently from 0570da9 to 43f2e7f Compare December 19, 2024 13:29
Tests randomly modify the style file.
This is not a problem for the final build, which uses
the cached version, but BundlMon was monitoring
a style file that was always different and did not
represent the true state.
In order not to be blocked, it will be delivered very soon.
BREAKING CHANGE: The date formatting function (`f` from `useI18n`) changes,
please refer to this doc.
https://date-fns.org/v2.30.0/docs/format
@Merkur39 Merkur39 force-pushed the feat/upgrade_date-fns branch from 43f2e7f to 9ad977c Compare December 19, 2024 13:48
@JF-Cozy
Copy link
Collaborator

JF-Cozy commented Dec 19, 2024

@JF-Cozy J'ai finalement converti la commit en BC, car le format de date change également. Il ne peut être automatiquement traité comme le type du premier argument.

Les BC côté date-fns ont des titres. Je pense qu'on devrait aider un peu ici et mettre dans le texte de notre BC, les BC auxquels faut faire référence dans la doc date-fns. Tous ne sont pas pertinent.

Ok je vais link uniquement la doc de format(en v2.30.0), puisque c'est ce que nous utilisons et le seul changement qui peut impacter les apps

on ne se comprend pas... c'est mieux de linker le changelog des BC plutôt que la doc. Mais tous les BC ne sont pas impactants dans l'app (car on ne met pas à jour date-fns dans une app, mais que dans certaines méthodes de cozy-ui) donc je n'aurai listé que les BC utiles pour l'app.

@Merkur39
Copy link
Member Author

@JF-Cozy J'ai finalement converti la commit en BC, car le format de date change également. Il ne peut être automatiquement traité comme le type du premier argument.

Les BC côté date-fns ont des titres. Je pense qu'on devrait aider un peu ici et mettre dans le texte de notre BC, les BC auxquels faut faire référence dans la doc date-fns. Tous ne sont pas pertinent.

Ok je vais link uniquement la doc de format(en v2.30.0), puisque c'est ce que nous utilisons et le seul changement qui peut impacter les apps

on ne se comprend pas... c'est mieux de linker le changelog des BC plutôt que la doc. Mais tous les BC ne sont pas impactants dans l'app (car on ne met pas à jour date-fns dans une app, mais que dans certaines méthodes de cozy-ui) donc je n'aurai listé que les BC utiles pour l'app.

Mais justement, les seuls BC utiles pour les apps sont des BC de la fonction format,et pas mal de ses fonctionnalités suivant l'usage dans l'app.
Il est plus clair d'avoir la page de cette doc je trouve (l'ayant vécu 😅 )

@Merkur39 Merkur39 merged commit a89c524 into master Dec 19, 2024
12 checks passed
@Merkur39 Merkur39 deleted the feat/upgrade_date-fns branch December 19, 2024 14:20
@cozy-bot
Copy link

🎉 This PR is included in version 115.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants