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

[TECH] Ajouter le workflow de prise en charge de dépréciation d'Ember.js sur Pix App #8159

Merged
merged 1 commit into from
Mar 5, 2024

Conversation

VincentHardouin
Copy link
Member

@VincentHardouin VincentHardouin commented Feb 23, 2024

🦄 Problème

Actuellement, il est compliqué de suivre les dépréciations d'Ember.js. Les nouvelles dépréciations sont juste des console.info et ne sont pas particulièrement suivies.

🤖 Proposition

Pour remédier à ça, Ember.js propose un workflow de prise en charge de dépréciation..

L'idée est :

  1. Installer la librairie ember-cli-deprecation-workflow
  2. Lancer les tests dans le navigateur
  3. A la fin des tests, lancer deprecationWorkflow.flushDeprecations()
  4. Copier le résultat dans config/deprecation-workflow.js

Une fois les dépréciations copiées dans le fichier, alors on peut activer le fait de jeter des erreurs pour une dépréciation spécifique pour la prendre en compte ET pour éviter que des devs utilisent l'ancienne méthode à nouveau sans faire exprès.

🌈 Remarques

Pour information, Ember.js a la volonté de rapatrier cette librairie directement dans le framework, pour que ça soit bénéfique au plus grand nombre

Autre information, nous avions déjà ce paquet fut un temps, mais il avait été retiré pour de mauvaise raison lors de la migration en Ember.js v4

💯 Pour tester

  • CI OK

@VincentHardouin VincentHardouin self-assigned this Feb 23, 2024
@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@laura-bergoens
Copy link
Member

c cool

Copy link
Contributor

@yaf yaf left a comment

Choose a reason for hiding this comment

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

🚀

@yaf yaf force-pushed the ember-data-deprecation-workflow branch from 743f4ed to 9d48522 Compare March 5, 2024 09:44
@pix-service-auto-merge pix-service-auto-merge merged commit 401e249 into dev Mar 5, 2024
7 of 8 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the ember-data-deprecation-workflow branch March 5, 2024 09:52
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 🚀 Ready to Merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants