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

[SERVICE PUBLIC]Update service public definition #421

Merged
merged 8 commits into from
Nov 15, 2024
Merged

Conversation

HAEKADI
Copy link
Contributor

@HAEKADI HAEKADI commented Nov 15, 2024

closes #416

Because dag_datalake_siren is not a package, Pytest cannot find import paths. the tests are commented for now awaiting code refactor.

@HAEKADI HAEKADI requested a review from hacherix as a code owner November 15, 2024 13:36
@HAEKADI HAEKADI self-assigned this Nov 15, 2024
@HAEKADI
Copy link
Contributor Author

HAEKADI commented Nov 15, 2024

@hacherix It's ready for review. Don't mind the unit tests 😅

@HAEKADI HAEKADI added the labels label Nov 15, 2024
Copy link
Contributor

@hacherix hacherix left a comment

Choose a reason for hiding this comment

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

LGTM!

@HAEKADI HAEKADI merged commit cff228f into main Nov 15, 2024
4 checks passed
@HAEKADI HAEKADI deleted the update-service-public branch November 15, 2024 15:05
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.

[Organisation] whitelisting de l'urssaf et blacklisting de la RATP
3 participants