bitcoin-dev-project / fresh-eyes

https://fresheyes.bitcoindevs.xyz
5 stars 3 forks source link

Most comments from original PR don't seem to be reflected in copied PR #44

Open harding opened 4 months ago

harding commented 4 months ago

I've tried two PRs with Fresh Eyes and very few of the code comments from the original PR seem to be reflected my local Fresh Eyes PR.

Steps to reproduce:

  1. Go to https://fresh-eyes.vercel.app and complete all the initial authorization steps
  2. Enter the following details: bitcoin / bitcoin / 29242
  3. Be redirected to https://github.com/harding/bitcoin/pull/2

What I see: my copied version of the PR only has two GitHub inline comments added, each of which refers to a short thread.

What I expect to see: references to at least the two dozen threads that exist on the original PR with its final commits. Ideally, several dozen more references to threads on earlier versions of the commits (although I acknowledge that's more difficult).

IgboPharaoh commented 3 months ago

Thank you very much for taking the time to use our product and for bringing this issue to our attention.

FreshEyes is designed to reproduce the pull request you're working on and aggregate all comments into different sections to streamline the review process:

Nevertheless I have taken note of the issue pointed out and also looked at your replicated version. I will replicate the pull request you mentioned to investigate this issue further and work on how we can better structure these comments based on your suggestions.

IgboPharaoh commented 2 months ago

Apologies this is coming late and we've not updated you on the progress of this issue.

Currently, we have a draft pull request aimed towards solving this but it has been put on hold as focus has been directed towards other projects. Please be rest assured this will be addressed in due time.