Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Feb 04 04:31
    Client build 5830 deployed
  • Feb 03 22:36
    pkaminski commented #915
  • Feb 03 22:29
    pkaminski labeled #997
  • Feb 03 21:22
    pkaminski labeled #999
  • Feb 03 21:22
    pkaminski opened #999
  • Feb 03 21:18
    pkaminski assigned #995
  • Feb 02 19:00
    pkaminski commented #997
  • Feb 02 19:00
    pkaminski labeled #998
  • Feb 02 16:25
    fahhem labeled #998
  • Feb 02 16:25
    fahhem labeled #998
  • Feb 02 16:25
    fahhem opened #998
  • Feb 02 16:21
    fahhem labeled #997
  • Feb 02 16:21
    fahhem labeled #997
  • Feb 02 16:21
    fahhem opened #997
  • Feb 02 04:11
    Client build 5826 deployed
  • Feb 02 04:02
    pkaminski unassigned #937
  • Feb 02 00:32
    fahhem commented #996
  • Feb 01 22:07
    pkaminski commented #996
  • Feb 01 22:06
    pkaminski assigned #996
  • Feb 01 22:06
    pkaminski labeled #996
Piotr Kaminski
@pkaminski
There's no way to do this right now, sorry. I agree it would be useful in longer multi-party reviews, though. Could you open an issue so we can track this? Thanks!
Misha Gusarov
@misha-ridge
sure, done
Deepak Chhikara
@deepak-harness
Hi Everyone
I'm facing issues in accessing reviewable. It keeps on showing the
resume session message But doesn't load the page. Look like issue with the session since i'm able to access it from the incognito. I tried logging out of github so that oauth can reset the session but no luck.
Is there a way to reset the site data so that it behaves in proper way
Piotr Kaminski
@pkaminski
Hey @deepak-harness, sorry about that. I've heard a few reports of this issue but never enough to get a repro. You can wipe out session information by clearing both local storage and IndexedDB for reviewable.io -- let me know if you need more detailed instructions.
Michael "Mike" Ferris
@Cellule
@pkaminski when you get a chance to take a look at Reviewable/Reviewable#950
it's currently preventing me and some of my colleagues from reviewing some PRs
Jeremy Nimmer
@jwnimmer-tri
The dashboard https://reviewable.io/reviews#- is failing to load for me as of the past couple hours. Failed to fetch a list of your reviews: TypeError: s.$ref.transaction is not a function Try again. FF 103.0 on Ubuntu 20, private window, no extensions. Any suggestions?
3 replies
Ryba
@sepatel
qq, I've been seeing this error for over an hour now. What does it mean? It is preventing me from doing reviews.
Failed to update your list of reviews: Error: GitHub error 403 on POST https://api.github.com/graphql: You have exceeded a secondary rate limit. Please wait a few minutes before you try again. Note that the review list refreshes automatically, so you don't need to reload the page manually. Try again.
Ryba
@sepatel
Also sometimes it is a 502 error instead of 403 error like in this case,
Failed to update your list of reviews: Error: GitHub error 502 on POST https://api.github.com/graphql: Something went wrong while executing your query. This may be the result of a timeout, or it could be a GitHub bug. Please includeBFEA:38C1:139041:4204FB:62F54A60when reporting this issue. Try again.
23 replies
Violet Whitney
@violetwhitney
Hi, I'm trying to track down invoices for my company's reviewable account but don't seem to have the option to see that detail in my account. Can someone please help?
3 replies
Misha Gusarov
@misha-ridge
@pkaminski Hey Piotr. Is there any chance to get back a single-click way to open a full history of a current conversation in Reviewable? This used to be a no-brainer, and now one has to understand which of 6 options is the right one, especially because I don't remember any need for 5 of 6: I've ever only needed "open the whole history of the current discussion".
27 replies
Misha Gusarov
@misha-ridge

@pkaminski I have noticed a weird issue: in some PRs source code does not get displayed (a spinner spins over every file in the PR). This persists after multiple refreshes.

I had a look at network panel, and haven't found any activity to fetch source code. Metadata (repository, labels, users, members etc) are fetched from GH properly.

The PR where I see it right now is https://reviewable.io/reviews/ridge/tectonic/16879

My browser is FF 106.0, and I'm under Linux/x86-64.

50 replies
Michał Kowalczyk
@mkow
is there any know outage happening right now? I'm getting "Unable to sign in: Firebase: A network AuthError (such as timeout, interrupted connection or unreachable host) has occurred. (auth/network-request-failed). " error on Reviewable and it loads the reviews, but without me being logged in
Michał Kowalczyk
@mkow
ok, disabling uMatrix fixes the issue, but can't use Reviewable with it being enabled, despite allowing everything
I've been using Reviewable with uMatrix for a few years already, did something change in Reviewable/GitHub auth process? or it's a change in uMatrix?
Piotr Kaminski
@pkaminski
No recent changes in Reviewable. Can't speak for Firebase, but I haven't heard of any changes on that side either. It's almost certainly a bug in uMatrix -- note that Reviewable's auth component runs in a shared worker process, so it's possible that uMatrix doesn't apply its settings properly there.
Michał Kowalczyk
@mkow
it worked yestarday, so it's probably a recent change in something
anyways, I have a local solution
Piotr Kaminski
@pkaminski
You can test the shared worker hypothesis by loading the page with ?debug=workers (which forces them to be non-shared) and seeing if that works.
Tony Novak
@afn
I just started using Reviewable and I love it already! Quick question: I have a PR that moved a frequently-imported JS file, so there are a lot of files in which the only change is to an import line. Is there a way to ignore changes that, say, match a certain regular expression?
1 reply
Piotr Kaminski
@pkaminski
older_comments.png
Alexey Feldgendler
@feldgendler
Hi! I remember that commit authors (pullRequest.revisions[i].commits[j].author) were added some time ago to the input for the custom completion function. Recently they have disappeared. Can we please have them back?
6 replies
Stefanus
@sdtqb
We're getting errors like Failed to publish: Request queued (action evaluate) but server did not respond using reviewable right now
2 replies
Alexey Feldgendler
@feldgendler
Hello! I'm getting an error message in a specific PR:
Error: Internal error: mergeable block range mismatch, last=442,444, next=358,358
9 replies
Stefanus
@sdtqb

I use stacked pull requests (no special tooling currently, I know there’s stuff like spr out there), i.e. I often have a chain of PRs where the bases are set so that A merges into main, B merges into A, C merges into B, etc.

My understanding[1] is that github is supposed to automatically retarget the dependent PRs, so e.g. if I merge “A”, the merge base for “B” would be set to main. But if I use “merge & delete” in reviewable to merge A into main, github seems to instead close the A<-B pull request. Any idea how to fix this?

[1] https://github.blog/changelog/2020-05-19-pull-request-retargeting/

6 replies
Øyvind Matheson Wergeland
@oyvindwe
Hi! We have a PR where the comments does not show up in Reviewable. They also don't show up as GitHub review conversation. The comments were posted from Reviewable, and are shown on the PR as one single comment from the reviewer, with links and quotes from all the commented files. This has maybe happened once before, but we don't have any steps to reproduce. Not sure how to handle this problem, so please advise.
2 replies
John Ian Bunag
@yvnbunag
Screenshot 2022-11-23 at 12.53.13.png
Hello :wave: , Reviewable seems to be down with the following error. ^
Piotr Kaminski
@pkaminski
Apologies, something must've gone terribly wrong with our last release. I just rolled back.
John Ian Bunag
@yvnbunag
That was quick, confirmed I can now access it, thanks!
Misha Gusarov
@misha-ridge
@pkaminski Hello Piotr. Is there a way we could adjust the limit of the discussion size that causes old comments to hide? We have a persistent problem that authors of PRs (who are not spending all their professional life in Reviewable) are confused by hiding beginnings of discussion threads as short as 3 messages. I'd say 10 messages in a thread would be a good threshold for our use-case.
4 replies
Stefanus
@sdtqb
is there some way to display source and destination branches in https://reviewable.io/reviews? Those are usually more meaningful to me than the PR numbers
6 replies
Stefanus
@sdtqb
we have an open PR that’s unreviewed but recognized by reviewable (it added its link to the description, etc.) and assigned to a reviewer 13 hours ago, but it’s not showing up on our Reviews tabs
1 reply
Sylvain Rabot
@sylr
Hi, I am trying out Reviewable for my organization and enabled it for a small set of repos but it turns out it is active for repo that is not supposed to be.
1 reply
image.png
This one for example is not supposed to be enabled but I just had a PR reviewable.
kyawzinthant
@Kyaw-Zin-Thant
hello everyone
How can I get code review for my personal public repo?
1 reply
dmitry-braid
@dmitry-braid
hi everyone! how can I open repo settings? according to docs https://docs.reviewable.io/repositories.html#repository-settings I should be able to click on repo's name, but it's not clickable for me
Piotr Kaminski
@pkaminski
Hey @dmitry-braid. Are you an admin for the repo in question?
dmitry-braid
@dmitry-braid
yes, I have admin access
Piotr Kaminski
@pkaminski
Strange. Is the repo's toggle enabled (whether it's on or off) -- ie, not greyed out?
dmitry-braid
@dmitry-braid
Screen Shot 2023-01-05 at 5.00.11 PM.png
Piotr Kaminski
@pkaminski
Are you talking about the "all current and future repos" item?
Or braid?
dmitry-braid
@dmitry-braid
toggle is off, reviews are not created autmatically
braid
maybe I should take a step back. when enabled, this edits PR description with review link. is it possible to add review link as a separate comment?
Piotr Kaminski
@pkaminski
Yes, that's in the repo settings, and you'd want to change the option before connecting the repo.
dmitry-braid
@dmitry-braid
ok, I'll try, thanks!