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

[BREAKING] 📦 Mise à jour des packages autour de SASS #745

Merged
merged 6 commits into from
Jan 13, 2025

Conversation

yaf
Copy link
Member

@yaf yaf commented Oct 23, 2024

💥 BREAKING_CHANGES

Remplacement de @import en @use , ( utilisation de @use as * dans les app afin d'éviter le doublon de namespace entre page/component ayant le même nom )

Déclaration des @use 'shadows' / 'typography' / 'fonts' / 'breakpoints' pour pouvoir utiliser les différentes variables/mixin associé

@use 'shadows'; 
@use 'typography' 
@use 'fonts'
@use 'breakpoints'

shadows.shadow()
fonts.$font-roboto
breakpoints.device-is()

Il n'est pas nécessaire de déclarer les uses pour les variables css native

🎄 Problème

Lors de la mise à jour de dépendances côté PixJunior, nous nous sommes retrouvés à mettre SASS à jour. Ça entraine quelque mise à jour SASS et du coup, des message d'alerte de dépréciation sur PixUI.

🎁 Proposition

Cette PR est une exploration de la mise à jour de ces paquets autour de SASS. L'idée est aussi de réfléchir à l'ordre dans lequel faire les mise à jours.

Faut-il commencer par mettre à jour les dépendances SASS sur les app, puis mettre à jour sur Pix UI et enfin, mettre à jour PixUI sur les app ?

🌟 Remarques

Des infos supplémentaires, trucs et astuces ?

🎅 Pour tester

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc.

@pix-bot-github
Copy link

Une fois l'application déployée, elle sera accessible à cette adresse https://ui-pr745.review.pix.fr
Les variables d'environnement seront accessibles sur scalingo https://dashboard.scalingo.com/apps/osc-fr1/pix-ui-review-pr745/environment

@yaf yaf self-assigned this Oct 23, 2024
@yaf yaf changed the title Upgrade packages [TECH] 📦 Mise à jour des packages autour de SASS Oct 23, 2024
@xav-car
Copy link
Contributor

xav-car commented Jan 9, 2025

Si ça force de changer la manière dont on utilises les Design Tokens "tout à fait" . pour le Breaking Change

@xav-car xav-car added cross-team Toutes les équipes de dev 🚧 Development in progress Work In Progress labels Jan 9, 2025
@yaf yaf marked this pull request as ready for review January 9, 2025 13:09
@xav-car xav-car changed the title [TECH] 📦 Mise à jour des packages autour de SASS [BREAKING] 📦 Mise à jour des packages autour de SASS Jan 9, 2025
@xav-car xav-car added the ⚠️ BREAKING CHANGES ⚠️ indique que la PR va introduire des breaking changes et donc qu'une version majeure est nécéssaire label Jan 9, 2025
@laura-bergoens
Copy link
Member

Une manière sinon de pas risquer le maxi breaking change serait de faire
@use 'mon/fichier' as *;
WDYT ?

@xav-car
Copy link
Contributor

xav-car commented Jan 10, 2025

Une manière sinon de pas risquer le maxi breaking change serait de faire @use 'mon/fichier' as *; WDYT ?

Le @use utilise comme namespace le nom du fichier.

il y aura incompatibilité si on utilise deux fois le même namespace ( component/campaign.scss / authenticated/campaign.scss ) . là il faut utiliser un as * ou un nom plus spécifique.

le passage de @import à @use requiert que là où on utilise des variables/mixins d'autres fichiers, de déclarer son @use.

De ce que j'ai compris 😅

@xav-car
Copy link
Contributor

xav-car commented Jan 10, 2025

Fonctionnel sur 1024pix/pix#11042

Copy link
Member

@laura-bergoens laura-bergoens left a comment

Choose a reason for hiding this comment

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

cool raoul

@pix-service-auto-merge pix-service-auto-merge merged commit 2f09242 into dev Jan 13, 2025
5 of 7 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the upgrade-packages branch January 13, 2025 10:03
pix-service-auto-merge pushed a commit that referenced this pull request Jan 13, 2025
# [53.0.0](v52.3.5...v53.0.0) (2025-01-13)

### 💥 BREAKING CHANGE

- [#745](#745) 📦 Mise à jour des packages autour de SASS
@pix-service-auto-merge
Copy link
Contributor

🎉 This PR is included in version 53.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
cross-team Toutes les équipes de dev released 🚀 Ready to Merge Tech review OK ⚠️ BREAKING CHANGES ⚠️ indique que la PR va introduire des breaking changes et donc qu'une version majeure est nécéssaire
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants