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: cz: only check commits in PR #80

Closed
wants to merge 2 commits into from
Closed

Conversation

kolyshkin
Copy link
Contributor

@kolyshkin kolyshkin commented Aug 5, 2024

Once something is merged, it does not make sense (and it's too late) to check it.

So, limit the check to commits in a given PR.

This fixes this repo CI (broken by merging #77).

Once something is merged, it does not make sense (and it's too late) to
check it.

So, limit the check to commits in a given PR.

Signed-off-by: Kir Kolyshkin <[email protected]>
Signed-off-by: Kir Kolyshkin <[email protected]>
@kolyshkin kolyshkin marked this pull request as ready for review August 5, 2024 20:20
@kolyshkin
Copy link
Contributor Author

No longer a draft. Tested that:

@kolyshkin kolyshkin changed the title [draft] ci/conv-commits: only check commits in PR ci: cz: only check commits in PR Aug 5, 2024
@polyfloyd
Copy link
Owner

This is great! Thanks!

@polyfloyd
Copy link
Owner

Cherry-picked the relevant commit

@polyfloyd polyfloyd closed this Aug 6, 2024
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