Added matchers for Result (#836)

[faogustavo]

Closes #836

---

Before making a review, please, go see the issue. I reported a problem there and it's blocking the PR.

[sksamuel]

I can't merge yet as it says branch is out of date.

[faogustavo]

I can't merge yet as it says branch is out of date.

Make sense merge if the tests are not passing by the problem that I reported on the issue?

[sksamuel]

I think it just means you need to rebase or something. Not quite sure. Try making another fork/branch and copying the changes in.

[faogustavo]

I think it just means you need to rebase or something. Not quite sure. Try making another fork/branch and copying the changes in.

Actually, this PR is up-to-date, but it's a draft. I was planning to remove the draft when I understand and solve the problem I reported on the issue. Have you seen my comment there?

[LeoColman]

There seem to be some failing tests tho.

Could you take a look at them?

[sksamuel]

I think it just means you need to rebase or something. Not quite sure. Try making another fork/branch and copying the changes in.

Actually, this PR is up-to-date, but it's a draft. I was planning to remove the draft when I understand and solve the problem I reported on the issue. Have you seen my comment there?

No I didn't see your comment ?

[sksamuel]

What is this waiting on? Do I need to do something?

[faogustavo]

We are waiting for the issue on kotlin to be solved. Please, see the issue from this task, #836 Everything is detailed there

[faogustavo]

I just pushed the workaround suggested by @sksamuel on the issue. We need to keep tracking the issue on kotlin issue tracker to solve this.

[sksamuel]

👍