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

ci(github): improvement to check config data #15406

Merged
merged 1 commit into from
Nov 18, 2024

Conversation

karliatto
Copy link
Member

@karliatto karliatto commented Nov 15, 2024

Description

Adding a little optimization to the check config data.
If a PR with the same hash (so same content) is already open it stops since there are not new changes to be added that the ones in that PR.
If there is a PR already open but the hash is different (so different content) it closes it, deletes the branch and create new one.

@karliatto karliatto force-pushed the fix/improvement-to-check-config-data branch from 93544ea to d93d520 Compare November 15, 2024 19:38
@karliatto karliatto changed the title wip: adding a TODO to remember wip: ci(github): improvement to check config data Nov 15, 2024
@karliatto karliatto force-pushed the fix/improvement-to-check-config-data branch from d93d520 to 2a0c580 Compare November 17, 2024 19:20
@karliatto karliatto marked this pull request as ready for review November 17, 2024 19:26
@karliatto karliatto requested a review from Nodonisko as a code owner November 17, 2024 19:26
@karliatto karliatto changed the title wip: ci(github): improvement to check config data ci(github): improvement to check config data Nov 17, 2024
@karliatto karliatto requested a review from mroz22 November 18, 2024 09:24
@karliatto karliatto merged commit 3578b27 into develop Nov 18, 2024
23 checks passed
@karliatto karliatto deleted the fix/improvement-to-check-config-data branch November 18, 2024 09:51
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