Skip to content
This repository has been archived by the owner on Jul 29, 2024. It is now read-only.

Skip merge request that reference no commit #92

Merged
merged 1 commit into from
Oct 12, 2022
Merged

Skip merge request that reference no commit #92

merged 1 commit into from
Oct 12, 2022

Conversation

psycofdj
Copy link
Contributor

This PR fixes a corner case where the check resource ends in error error running command: SHA must be a non-empty string when the target projet has a merge-request that references no commit (sha == null in gitlab api).

It also remove a dead condition, err is tested twice for nil in a row.

@samcontesse
Copy link
Owner

Thanks for your contribution @psycofdj

@samcontesse samcontesse merged commit a88742b into samcontesse:master Oct 12, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants