internal/lsp: use x/xerrors to create new errors #108

jan-xyz · 2019-05-31T22:30:06Z

This relates to golang/go#31374 and should switch all instances within gopls to use x/errors instead of fmt to create new errors.

gopherbot · 2019-05-31T22:32:06Z

This PR (HEAD: 76f8316) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/tools/+/179880 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

gopherbot · 2019-05-31T22:33:51Z

Message from Gobot Gobot:

Patch Set 1:

Congratulations on opening your first change. Thank you for your contribution!

Next steps:
Within the next week or so, a maintainer will review your change and provide
feedback. See https://golang.org/doc/contribute.html#review for more info and
tips to get your patch through code review.

Most changes in the Go project go through a few rounds of revision. This can be
surprising to people new to the project. The careful, iterative review process
is our way of helping mentor contributors and ensuring that their contributions
have a lasting impact.

During May-July and Nov-Jan the Go project is in a code freeze, during which
little code gets reviewed or merged. If a reviewer responds with a comment like
R=go1.11, it means that this CL will be reviewed as part of the next development
cycle. See https://golang.org/s/release for more details.

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-05-31T22:38:04Z

This PR (HEAD: de31d86) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/tools/+/179880 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

gopherbot · 2019-05-31T22:41:08Z

This PR (HEAD: 3706d2b) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/tools/+/179880 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

gopherbot · 2019-05-31T22:46:32Z

This PR (HEAD: 53e600e) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/tools/+/179880 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

jan-xyz · 2019-05-31T22:58:39Z

After filing this PR I am wondering if that’s what was asked by the ticket? Should the logger also now log something additionally?

I don't quite understand how the transition to go1.13 will behave, the documentation states most of the functionality will be absorbed into the standard lib. Will that make a transition back to fmt necessary in the future?

gopherbot · 2019-07-02T15:43:57Z

Message from Rebecca Stambler:

Patch Set 6:

Thank you for opening this PR! Sorry about the delay in reviewing - there is a bug with the Github/Gerrit integration that prevents us from automatically adding reviewers. We'll take a look at it ASAP.

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-07-02T17:12:37Z

Message from Ian Cottrell:

Uploaded patch set 7: Run-TryBot+1.

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-07-02T17:13:28Z

Message from Gobot Gobot:

Patch Set 7:

TryBots beginning. Status page: https://farmer.golang.org/try?commit=a13e7dd0

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-07-02T17:16:00Z

Message from Gobot Gobot:

Patch Set 7:

Build is still in progress...
This change failed on linux-amd64 (Go 1.11.x):
See https://storage.googleapis.com/go-build-log/f0569288/linux-amd64_c70375e8.log

Other builds still in progress; subsequent failure notices suppressed until final report. Consult https://build.golang.org/ to see whether they are new failures. Keep in mind that TryBots currently test exactly your git commit, without rebasing. If your commit's git parent is old, the failure might've already been fixed.

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-07-02T17:18:50Z

Message from Gobot Gobot:

Patch Set 7: TryBot-Result-1

Consult https://build.golang.org/ to see whether they are new failures. Keep in mind that TryBots currently test exactly your git commit, without rebasing. If your commit's git parent is old, the failure might've already been fixed.

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-07-02T17:20:29Z

Message from Gobot Gobot:

Patch Set 8:

TryBots beginning. Status page: https://farmer.golang.org/try?commit=47800801

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-07-02T17:23:07Z

Message from Gobot Gobot:

Patch Set 8:

Build is still in progress...
This change failed on freebsd-amd64-12_0:
See https://storage.googleapis.com/go-build-log/e62c3d35/freebsd-amd64-12_0_26bd5c74.log

Other builds still in progress; subsequent failure notices suppressed until final report. Consult https://build.golang.org/ to see whether they are new failures. Keep in mind that TryBots currently test exactly your git commit, without rebasing. If your commit's git parent is old, the failure might've already been fixed.

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-07-02T17:26:29Z

Message from Gobot Gobot:

Patch Set 8: TryBot-Result-1

Consult https://build.golang.org/ to see whether they are new failures. Keep in mind that TryBots currently test exactly your git commit, without rebasing. If your commit's git parent is old, the failure might've already been fixed.

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-07-06T12:04:39Z

This PR (HEAD: fc35060) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/tools/+/179880 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

gopherbot · 2019-07-06T12:07:13Z

Message from Jan Steinke:

Patch Set 9:

Run-TryBot+1.

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-07-06T12:34:55Z

This PR (HEAD: 2fe67fd) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/tools/+/179880 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

gopherbot · 2019-07-06T12:42:12Z

Message from Jan Steinke:

Patch Set 9:

No problem Rebecca :)

I rebased my changes onto the latest state of master and fixed a few issues that popped up due to the changes. However, there is now one thing that I cannot understand. The lsp_test fails

2019/07/06 14:23:20 Info: no signature help for file:///var/folders/6j/2n60g37s0f94qxz28gfx8fn40000gn/T/TestLSP_GOPATH003021802/lsp/src/golang.org/x/tools/internal/lsp/signature/signature.go:64:2 : no signature help within a function declaration
2019/07/06 14:23:31 Info: no signature help for file:///var/folders/6j/2n60g37s0f94qxz28gfx8fn40000gn/T/TestLSP_Modules846208577/primarymod/lsp/signature/signature.go:64:2 : no signature help within a function declaration
--- FAIL: TestBytesOffset (0.00s)
    lsp_test.go:830: want -1 for "aaa\nbbb\n"(Line:0,Character:4), but got 3
    lsp_test.go:830: want -1 for "aaa\nbbb\n"(Line:1,Character:4), but got 7
FAIL
FAIL	golang.org/x/tools/internal/lsp	20.126s

As this is an error case with an invalid point, I assume it is due to the move to xerrors and some matching onto the error message or something that is failing now. I don't see though where this is coming from. Do you have a hint where to look?

Also: I was wondering if it might be smart to alias the package to errors so that it is easier to switch to the stdlib implementation in go 1.13

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-07-06T12:48:55Z

Message from Jan Steinke:

Patch Set 10:

One more thing: I only changed the errors in internal/lsp, do you think it's smart to do that as a first step, or should all errors be switched to xerrors?

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-07-06T16:32:14Z

Message from Jan Steinke:

Patch Set 10:

Patch Set 9:

No problem Rebecca :)

I rebased my changes onto the latest state of master and fixed a few issues that popped up due to the changes. However, there is now one thing that I cannot understand. The lsp_test fails
2019/07/06 14:23:20 Info: no signature help for file:///var/folders/6j/2n60g37s0f94qxz28gfx8fn40000gn/T/TestLSP_GOPATH003021802/lsp/src/golang.org/x/tools/internal/lsp/signature/signature.go:64:2 : no signature help within a function declaration
2019/07/06 14:23:31 Info: no signature help for file:///var/folders/6j/2n60g37s0f94qxz28gfx8fn40000gn/T/TestLSP_Modules846208577/primarymod/lsp/signature/signature.go:64:2 : no signature help within a function declaration
--- FAIL: TestBytesOffset (0.00s)
    lsp_test.go:830: want -1 for "aaa\nbbb\n"(Line:0,Character:4), but got 3
    lsp_test.go:830: want -1 for "aaa\nbbb\n"(Line:1,Character:4), but got 7
FAIL
FAIL	golang.org/x/tools/internal/lsp	20.126s
As this is an error case with an invalid point, I assume it is due to the move to xerrors and some matching onto the error message or something that is failing now. I don't see though where this is coming from. Do you have a hint where to look?

Also: I was wondering if it might be smart to alias the package to errors so that it is easier to switch to the stdlib implementation in go 1.13

I noticed that these tests also fail on master when I run 'em locally. Is there something special about these tests?

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-07-08T19:28:07Z

Message from Rebecca Stambler:

Patch Set 10:

Patch Set 10:
Patch Set 9:

No problem Rebecca :)

I rebased my changes onto the latest state of master and fixed a few issues that popped up due to the changes. However, there is now one thing that I cannot understand. The lsp_test fails
2019/07/06 14:23:20 Info: no signature help for file:///var/folders/6j/2n60g37s0f94qxz28gfx8fn40000gn/T/TestLSP_GOPATH003021802/lsp/src/golang.org/x/tools/internal/lsp/signature/signature.go:64:2 : no signature help within a function declaration
2019/07/06 14:23:31 Info: no signature help for file:///var/folders/6j/2n60g37s0f94qxz28gfx8fn40000gn/T/TestLSP_Modules846208577/primarymod/lsp/signature/signature.go:64:2 : no signature help within a function declaration
--- FAIL: TestBytesOffset (0.00s)
    lsp_test.go:830: want -1 for "aaa\nbbb\n"(Line:0,Character:4), but got 3
    lsp_test.go:830: want -1 for "aaa\nbbb\n"(Line:1,Character:4), but got 7
FAIL
FAIL	golang.org/x/tools/internal/lsp	20.126s
As this is an error case with an invalid point, I assume it is due to the move to xerrors and some matching onto the error message or something that is failing now. I don't see though where this is coming from. Do you have a hint where to look?

Also: I was wondering if it might be smart to alias the package to errors so that it is easier to switch to the stdlib implementation in go 1.13
I noticed that these tests also fail on master when I run 'em locally. Is there something special about these tests?

The test failures were due to https://go-review.googlesource.com/c/tools/+/185058 and fixed in https://go-review.googlesource.com/c/tools/+/185220. Sorry about that! Do you mind rebasing one more time?

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-07-08T20:58:19Z

This PR (HEAD: 45f52f7) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/tools/+/179880 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

gopherbot · 2019-07-08T21:10:26Z

Message from Jan Steinke:

Patch Set 11:

The test failures were due to https://go-review.googlesource.com/c/tools/+/185058 and fixed in https://go-review.googlesource.com/c/tools/+/185220. Sorry about that! Do you mind rebasing one more time?

Not at all :) Done!

It's basically a race against you guys. There are new fmt.Errorf that come in with each new merge. So excuse me for not converting all.

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-07-08T21:19:52Z

This PR (HEAD: 6e5686f) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/tools/+/179880 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

gopherbot · 2019-07-08T21:39:52Z

Message from Rebecca Stambler:

Patch Set 12: Run-TryBot+1

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-07-08T21:39:53Z

Message from Gobot Gobot:

Patch Set 12:

TryBots beginning. Status page: https://farmer.golang.org/try?commit=c0c67f7f

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-07-08T21:42:45Z

Message from Gobot Gobot:

Patch Set 12:

Build is still in progress...
This change failed on freebsd-amd64-12_0:
See https://storage.googleapis.com/go-build-log/a19c0ced/freebsd-amd64-12_0_39c962de.log

Other builds still in progress; subsequent failure notices suppressed until final report. Consult https://build.golang.org/ to see whether they are new failures. Keep in mind that TryBots currently test exactly your git commit, without rebasing. If your commit's git parent is old, the failure might've already been fixed.

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-07-08T21:45:11Z

Message from Gobot Gobot:

Patch Set 12: TryBot-Result-1

Consult https://build.golang.org/ to see whether they are new failures. Keep in mind that TryBots currently test exactly your git commit, without rebasing. If your commit's git parent is old, the failure might've already been fixed.

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-07-08T21:49:06Z

Message from Rebecca Stambler:

Patch Set 12: -Run-TryBot

These errors are probably related to https://go-review.googlesource.com/c/xerrors/+/184937, so that CL will need to be submitted before this one.

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-07-09T18:59:43Z

This PR (HEAD: db0cab5) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/tools/+/179880 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

gopherbot · 2019-07-09T19:09:01Z

Message from Rebecca Stambler:

Patch Set 13:

(1 comment)

Sorry about the delay here. This CL will have to be blocked by golang/go#32246.

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-07-10T19:39:20Z

This PR (HEAD: 6bfd3e5) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/tools/+/179880 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

gopherbot · 2019-08-04T14:44:59Z

This PR (HEAD: 2cf133f) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/tools/+/179880 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

gopherbot · 2019-08-05T20:13:13Z

Message from Rebecca Stambler:

Patch Set 16: Run-TryBot+1 Code-Review+2

Looks like there is still a merge conflict here. Can you please resolve it, Jan? Thanks!

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-08-05T20:13:14Z

Message from Gobot Gobot:

Patch Set 16:

TryBots beginning. Status page: https://farmer.golang.org/try?commit=0f797337

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-08-05T20:14:14Z

Message from Gobot Gobot:

Patch Set 16:

Build is still in progress...
This change failed on freebsd-amd64-12_0:
See https://storage.googleapis.com/go-build-log/d1338ec3/freebsd-amd64-12_0_adff9fa6.log

Other builds still in progress; subsequent failure notices suppressed until final report. Consult https://build.golang.org/ to see whether they are new failures. Keep in mind that TryBots currently test exactly your git commit, without rebasing. If your commit's git parent is old, the failure might've already been fixed.

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-08-05T20:16:15Z

Message from Gobot Gobot:

Patch Set 16: TryBot-Result-1

Consult https://build.golang.org/ to see whether they are new failures. Keep in mind that TryBots currently test exactly your git commit, without rebasing. If your commit's git parent is old, the failure might've already been fixed.

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-08-06T19:13:16Z

This PR (HEAD: f4a55d9) has been imported to Gerrit for code review.

Please visit https://go-review.googlesource.com/c/tools/+/179880 to see it.

Tip: You can toggle comments from me using the comments slash command (e.g. /comments off)
See the Wiki page for more info

gopherbot · 2019-08-06T19:16:25Z

Message from Jan Steinke:

Patch Set 17:

The repository is quite fast moving. I rebased again and fixed the conflicts.

Patch Set 16: Run-TryBot+1 Code-Review+2

Looks like there is still a merge conflict here. Can you please resolve it, Jan? Thanks!

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-08-06T19:26:28Z

Message from Rebecca Stambler:

Patch Set 17: Run-TryBot+1

Thanks!

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-08-06T19:26:29Z

Message from Gobot Gobot:

Patch Set 17:

TryBots beginning. Status page: https://farmer.golang.org/try?commit=a1b442f0

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-08-06T19:29:36Z

Message from Gobot Gobot:

Patch Set 17:

Build is still in progress...
This change failed on linux-amd64 (Go 1.11.x):
See https://storage.googleapis.com/go-build-log/7a853b65/linux-amd64_dcd0ddee.log

Other builds still in progress; subsequent failure notices suppressed until final report. Consult https://build.golang.org/ to see whether they are new failures. Keep in mind that TryBots currently test exactly your git commit, without rebasing. If your commit's git parent is old, the failure might've already been fixed.

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-08-06T19:33:12Z

Message from Gobot Gobot:

Patch Set 17: TryBot-Result-1

1 of 10 TryBots failed:
Failed on linux-amd64 (Go 1.11.x): https://storage.googleapis.com/go-build-log/7a853b65/linux-amd64_dcd0ddee.log

Consult https://build.golang.org/ to see whether they are new failures. Keep in mind that TryBots currently test exactly your git commit, without rebasing. If your commit's git parent is old, the failure might've already been fixed.

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-08-06T19:37:39Z

Message from Rebecca Stambler:

Patch Set 18: Run-TryBot+1

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-08-06T19:37:40Z

Message from Gobot Gobot:

Patch Set 18:

TryBots beginning. Status page: https://farmer.golang.org/try?commit=0b3ec735

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-08-06T19:40:58Z

Message from Gobot Gobot:

Patch Set 18:

Build is still in progress...
This change failed on linux-amd64 (Go 1.11.x):
See https://storage.googleapis.com/go-build-log/7a853b65/linux-amd64_f973c0d1.log

Other builds still in progress; subsequent failure notices suppressed until final report. Consult https://build.golang.org/ to see whether they are new failures. Keep in mind that TryBots currently test exactly your git commit, without rebasing. If your commit's git parent is old, the failure might've already been fixed.

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

gopherbot · 2019-08-06T19:43:56Z

Message from Gobot Gobot:

Patch Set 18: TryBot-Result-1

1 of 10 TryBots failed:
Failed on linux-amd64 (Go 1.11.x): https://storage.googleapis.com/go-build-log/7a853b65/linux-amd64_f973c0d1.log

Consult https://build.golang.org/ to see whether they are new failures. Keep in mind that TryBots currently test exactly your git commit, without rebasing. If your commit's git parent is old, the failure might've already been fixed.

Please don’t reply on this GitHub thread. Visit golang.org/cl/179880.
After addressing review feedback, remember to publish your drafts!

This relates to golang/go#31374 and should switch all instances within `gopls` to use `x/errors` instead of `fmt` to create new errors. Change-Id: I18339b75d12418d852e0dcc2ba0ed6c2970783b3 GitHub-Last-Rev: f4a55d9 GitHub-Pull-Request: #108 Reviewed-on: https://go-review.googlesource.com/c/tools/+/179880 Run-TryBot: Rebecca Stambler <[email protected]> Reviewed-by: Rebecca Stambler <[email protected]>

gopherbot · 2019-08-06T19:48:52Z

This PR is being closed because golang.org/cl/179880 has been merged.

This relates to golang/go#31374 and should switch all instances within `gopls` to use `x/errors` instead of `fmt` to create new errors. Change-Id: I18339b75d12418d852e0dcc2ba0ed6c2970783b3 GitHub-Last-Rev: f4a55d9b79e7458ef1f1e06cb5eabbabd884f321 GitHub-Pull-Request: golang/tools#108 Reviewed-on: https://go-review.googlesource.com/c/tools/+/179880 Run-TryBot: Rebecca Stambler <[email protected]> Reviewed-by: Rebecca Stambler <[email protected]>

googlebot added the cla: yes label May 31, 2019

jan-xyz changed the title ~~use x/xerrors to create new errors~~ internal/lsp: use x/xerrors to create new errors May 31, 2019

gopherbot force-pushed the master branch from d303ba2 to ce2cddb Compare June 11, 2019 20:13

use x/xerrors to create new errors

f4a55d9

gopherbot closed this Aug 6, 2019

jan-xyz deleted the xerror branch August 8, 2019 18:48

internal/lsp: use x/xerrors to create new errors #108

internal/lsp: use x/xerrors to create new errors #108

Conversation

jan-xyz commented May 31, 2019 • edited Loading

gopherbot commented May 31, 2019

gopherbot commented May 31, 2019

gopherbot commented May 31, 2019

gopherbot commented May 31, 2019

gopherbot commented May 31, 2019

jan-xyz commented May 31, 2019

gopherbot commented Jul 2, 2019

gopherbot commented Jul 2, 2019

gopherbot commented Jul 2, 2019

gopherbot commented Jul 2, 2019

gopherbot commented Jul 2, 2019

gopherbot commented Jul 2, 2019

gopherbot commented Jul 2, 2019

gopherbot commented Jul 2, 2019

gopherbot commented Jul 6, 2019

gopherbot commented Jul 6, 2019

gopherbot commented Jul 6, 2019

gopherbot commented Jul 6, 2019

gopherbot commented Jul 6, 2019

gopherbot commented Jul 6, 2019

gopherbot commented Jul 8, 2019

gopherbot commented Jul 8, 2019

gopherbot commented Jul 8, 2019

gopherbot commented Jul 8, 2019

gopherbot commented Jul 8, 2019

gopherbot commented Jul 8, 2019

gopherbot commented Jul 8, 2019

gopherbot commented Jul 8, 2019

gopherbot commented Jul 8, 2019

gopherbot commented Jul 9, 2019

gopherbot commented Jul 9, 2019

gopherbot commented Jul 10, 2019

gopherbot commented Aug 4, 2019

gopherbot commented Aug 5, 2019

gopherbot commented Aug 5, 2019

gopherbot commented Aug 5, 2019

gopherbot commented Aug 5, 2019

gopherbot commented Aug 6, 2019

gopherbot commented Aug 6, 2019

gopherbot commented Aug 6, 2019

gopherbot commented Aug 6, 2019

gopherbot commented Aug 6, 2019

gopherbot commented Aug 6, 2019

gopherbot commented Aug 6, 2019

gopherbot commented Aug 6, 2019

gopherbot commented Aug 6, 2019

gopherbot commented Aug 6, 2019

gopherbot commented Aug 6, 2019

jan-xyz commented May 31, 2019 •

edited

Loading