mrhlpr: mr: use a simpler mechanism to detect if user approved a MR

This doesn't depend on an auth error to detect if a user has approved a MR, and fixes #24 (closed).

For some reason mr.approval_state().get was 404'ing, causing #24 (closed).

approved this merge request

Thanks for the fix! LGTM

requested review from @Newbyte

assigned to @pabloyoyoista

Thanks for investigating this!

added 1 commit

• a8b57d2e - mrhlpr: mr: use a simpler mechanism to detect if user approved a MR

Compare with previous version

added 1 commit

• 79be226d - mrhlpr: mr: use a simpler mechanism to detect if user approved a MR

Compare with previous version

approved this merge request

This fixed #24 (closed) for me

resolved all threads

I still get failures setting auto merge, so this does not seem to consistently fix #24 (closed).

resolved all threads

approved this merge request

added 4 commits

• 79be226d...d49a1ccf - 3 commits from branch master
• 297ff40a - mrhlpr: mr: use a simpler mechanism to detect if user approved a MR (MR 60)

Compare with previous version

merged