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

Improve the CI to allow adding new config #11341

Open
pascalgrimaud opened this issue Nov 11, 2024 · 1 comment · May be fixed by #11352
Open

Improve the CI to allow adding new config #11341

pascalgrimaud opened this issue Nov 11, 2024 · 1 comment · May be fixed by #11352

Comments

@pascalgrimaud
Copy link
Member

Following discussion in

We need to improve our CI to allow adding new config.
The workflow should be:

If the app already exists in main branch:
- get md5sum of app in main branch
- get md5sum of app in pull request branch
- compare
    - if it's the same value, stop the CI
    - if it's a different value, launch tests

If the app doesn't exist in main branch
- get md5sum of app in main branch, not exist
- no need to get md5sum of app in pull request branch
- nothing to compare, launch tests
@renanfranca
Copy link
Contributor

I implemented a solution, and I am going to try it out tomorrow morning 👍

Thanks for the complete description of what I need to do.

@renanfranca renanfranca linked a pull request Nov 12, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants