omp packages

pi-feedback0.1.0

Give Pi feedback on its changes: comment on lines in a GitHub-style browser view with difit and send them back with /feedback

pringled Published 7 days ago MIT ~455 downloads/mo

Untested on omp

This is a pi package. No one has reported whether it works on omp yet. If you try it, add your result below.

Did it work for you?

Result

One report per person. A new report replaces your old one.

Reports 0

No reports yet.

Readme

pi-feedback

Give Pi feedback on its changes: look through them in a GitHub-style view in your browser, leave comments on lines, and send them all back to Pi in one go.

pi-feedback demo
pi install npm:pi-feedback

This also installs difit, which draws the review page in your browser (about 235 MB with its own dependencies).

Usage

Run /feedback. Your browser opens with everything on the current branch since it left main, including uncommitted changes and new files. Click a line, or drag over several, to comment:

Pi's changes in difit, with two comments on the diff

When you're done, press Send to Pi in the bottom-right corner, or run /feedback again in Pi. Your comments go to Pi as one message:

Here's my feedback on the changes against main:

stats.py:L14-L20
Use statistics.median here instead of writing our own

test_stats.py:L14
Also test an even number of values

If Pi is busy, the message waits until it's done. If you didn't leave any comments, nothing is sent. Closing or reloading the tab doesn't end it, and the view updates by itself when files change, so you can keep it open while Pi works.

  • /feedback dev compares with another branch instead of the default one.
  • On main itself, /feedback shows your uncommitted changes.
  • If the browser didn't open, for example because Pi runs in a sandbox, the address is shown above Pi's input.

License

MIT