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] Déplace YamlParsingError vers les erreurs partagées de src #9584

Merged
merged 1 commit into from
Jul 19, 2024

Conversation

yaf
Copy link
Contributor

@yaf yaf commented Jul 18, 2024

🦄 Problème

cf https://github.com/1024pix/pix/blob/dev/docs/adr/0051-nouvelle-arborescence-api.md

🤖 Proposition

Déplacer YamlParsingError vers src/shared/domain/errors.js

🌈 Remarques

💯 Pour tester

@yaf yaf added this to the Bounded context migration milestone Jul 18, 2024
@yaf yaf self-assigned this Jul 18, 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 :

@yaf yaf changed the title refactor(api) move YamlParsingError to src [TECH] Déplace YamlParsingError vers les erreurs partagées de src Jul 18, 2024
@yaf yaf force-pushed the migrate-yaml-parsing-error-to-src branch 3 times, most recently from f74b46e to 515d760 Compare July 18, 2024 14:37
notes Outdated Show resolved Hide resolved
@yaf yaf force-pushed the migrate-yaml-parsing-error-to-src branch 2 times, most recently from f7267af to 7c0e59c Compare July 19, 2024 07:47
@yaf yaf marked this pull request as draft July 19, 2024 07:59
@yaf yaf force-pushed the migrate-yaml-parsing-error-to-src branch from 7c0e59c to 744be12 Compare July 19, 2024 08:31
@yaf yaf marked this pull request as ready for review July 19, 2024 08:45
@dlahaye
Copy link
Contributor

dlahaye commented Jul 19, 2024

Lundi on devrait avoir un outil pour tester plus finement ce genre de cas, avec l'aide de Romain. Et on pourra réutiliser cet outil par la suite.

@yaf yaf force-pushed the migrate-yaml-parsing-error-to-src branch from 744be12 to 37512d2 Compare July 19, 2024 14:26
@pix-service-auto-merge pix-service-auto-merge merged commit bd2d486 into dev Jul 19, 2024
6 of 7 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the migrate-yaml-parsing-error-to-src branch July 19, 2024 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants