-
-
Notifications
You must be signed in to change notification settings - Fork 50
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
Failed to install 1.0.0 and run it ! #195
Comments
this error is likely caused by a field required, but was null. exactly @SerialName("approvals_before_merge")
val approvalsBeforeMerge: Int = 0, we default the approvalsBeforeMerge to 0 if the field is not present. |
@gianluz Yes I can try Nevertheless I can prepare a PR Could you elaborate what to do ?
|
@gianluz As I can't easily debug/test tell me if : you need:
Or
|
Hi @gianluz, I'm still stuck with my issue on my computer, but can you please put me in the good direction that I push a PR. |
I updated the script on a specific branch to be able to install the fix on Bitrise, but is seems to doesn't work yet !
@gianluz any idea ? I already saw some issue about date, wasn't fixed ? |
@gianluz I succeed update my laptop ! |
When we look at the documentation https://docs.gitlab.com/ee/api/merge_requests.html, it seems there are some gap with the json file that is used in the project. Could someone confirm ? |
to target a specific Dangerfile you can use |
i haven't verified personally but DangerKotlin maps data coming back from danger-js, the model we need to map is the one coming from danger-js and not the one coming from the gitlab api. @f-meloni correct me if i'm wrong |
You can use |
to get the whole json that Danger Kotlin receives, you can use |
|
I was not able to install previous version, but here is the log of the script that install and run danger-kt on my ci
I'm running Danger on Bitrise with a Linux stack
The script that run Danger is:
My DangerFile is:
The log:
The text was updated successfully, but these errors were encountered: