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/Action github conditionnelle #32

Merged
merged 13 commits into from
Feb 13, 2024
Merged

Conversation

saveliy-sviridov
Copy link
Contributor

Ajout de condition d'exécution de l'action generate-model vérifiant si des changements ont été apportés aux fichiers *.py ou *.xlsx

@saveliy-sviridov saveliy-sviridov changed the title Feat/Action github conditionnelle WIP: Feat/Action github conditionnelle Feb 1, 2024
Copy link

github-actions bot commented Feb 1, 2024

There is no coverage information present for the Files changed

Total Project Coverage 64.66% 🍏

Copy link

github-actions bot commented Feb 1, 2024

There is no coverage information present for the Files changed

Total Project Coverage 64.66% 🍏

Copy link

github-actions bot commented Feb 1, 2024

There is no coverage information present for the Files changed

Total Project Coverage 64.66% 🍏

Copy link

github-actions bot commented Feb 1, 2024

There is no coverage information present for the Files changed

Total Project Coverage 64.66% 🍏

@saveliy-sviridov saveliy-sviridov changed the title WIP: Feat/Action github conditionnelle Feat/Action github conditionnelle Feb 2, 2024
Copy link

github-actions bot commented Feb 5, 2024

There is no coverage information present for the Files changed

Total Project Coverage 64.66% 🍏

Copy link
Collaborator

@romainfd romainfd left a comment

Choose a reason for hiding this comment

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

A confirmer

@@ -49,6 +60,7 @@ jobs:
run: sudo apt install -y clang-format

- name: Generate Java classes
if: steps.filter.outputs.parsing_required == 'true'
Copy link
Collaborator

Choose a reason for hiding this comment

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

En fait, à voir avec @bou3108, mais il faut peut-être garder ça et le cp à chaque fois (car si on change le template notamment, on veut que ça regénère) !
Comme c'est pas bruyant (si pas de changements, ça fait rien alors que le csv_parser fait des schémas avec la date + en docx / pdf donc tjs mis à jour)

Copy link

github-actions bot commented Feb 9, 2024

There is no coverage information present for the Files changed

Total Project Coverage 64.66% 🍏

Copy link
Collaborator

@romainfd romainfd left a comment

Choose a reason for hiding this comment

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

Amazing!

@romainfd romainfd merged commit 40080a2 into main Feb 13, 2024
1 check passed
@romainfd romainfd deleted the feat/conditional-github-action branch February 13, 2024 15:16
Copy link

There is no coverage information present for the Files changed

Total Project Coverage 64.66% 🍏

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

Successfully merging this pull request may close these issues.

2 participants