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

fix(xgenny): run all dry runners before the wet run in the xgenny pkg #4000

Merged
merged 2 commits into from
Mar 5, 2024

Conversation

Pantani
Copy link
Collaborator

@Pantani Pantani commented Mar 4, 2024

Description

Now, we run a dry run, and after the wet run, for each modification. We should run the dry runner before all modifications so we can run the wet runner.

@Pantani Pantani self-assigned this Mar 4, 2024
@github-actions github-actions bot added component:ci CI/CD workflow and automated jobs. component:configs component:packages labels Mar 4, 2024
@Pantani Pantani changed the title fix: run all dry runners before the we runners for the xgenny pkg Run all dry runners before the wet run in the xgenny pkg Mar 4, 2024
@Pantani Pantani changed the title Run all dry runners before the wet run in the xgenny pkg fix(xgenny): run all dry runners before the wet run in the xgenny pkg Mar 4, 2024
@Pantani Pantani merged commit bd734bb into main Mar 5, 2024
47 checks passed
@Pantani Pantani deleted the fix/run-dry-run branch March 5, 2024 16:34
@julienrbrt
Copy link
Member

I am not super familiar with this part of the codebase @Pantani, but do you suggest this is something that should get backported or we leave it for v29?

@salmad3
Copy link
Member

salmad3 commented Mar 11, 2024

Would be good to backport imo.

@Pantani
Copy link
Collaborator Author

Pantani commented Mar 11, 2024

I think is not necessary, we can hold for the 29 if we are going to release it soon

julienrbrt pushed a commit that referenced this pull request May 29, 2024
…kg (#4000)

* run all dry runners before the we runners for the xgenny pkg

* add changelog

---------

Co-authored-by: Pantani <Pantani>
@Pantani Pantani added the backport/v28.x.y Backport to v28.x.y label Jul 2, 2024
mergify bot pushed a commit that referenced this pull request Jul 2, 2024
…kg (#4000)

* run all dry runners before the we runners for the xgenny pkg

* add changelog

---------

Co-authored-by: Pantani <Pantani>
(cherry picked from commit bd734bb)
@Pantani Pantani removed the backport/v28.x.y Backport to v28.x.y label Jul 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants