Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Review comments support #75

Open
itainoam opened this issue Jan 16, 2019 · 15 comments
Open

Review comments support #75

itainoam opened this issue Jan 16, 2019 · 15 comments
Labels
enhancement New feature or request

Comments

@itainoam
Copy link

itainoam commented Jan 16, 2019

Does forge currently support review comments (comments on a portion of the diff)?

If not, is there a plan to add it in the future?

@itainoam itainoam changed the title Review Comments Review comments support Jan 16, 2019
@tarsius
Copy link
Member

tarsius commented Jan 16, 2019

No. Yes.

@tarsius tarsius added the enhancement New feature or request label Jan 16, 2019
@titaniumbones
Copy link

Ah man I'm so totally looking forward to this! Shout to starting marking 50 student assignments a week via code review :-/

@leungbk
Copy link

leungbk commented Sep 15, 2019

For the time being, this seems to be what you're looking for and can integrate with Forge: https://github.com/charignon/github-review

@nshi
Copy link

nshi commented Sep 30, 2019

This would be a great feature. Is there any plan for adding it soon?

@tarsius
Copy link
Member

tarsius commented Sep 30, 2019

Yes it would, but no.

@Fuuzetsu
Copy link

Fuuzetsu commented Oct 5, 2019

I don't suppose there's a way to make this issue prioritized, like financial donations &c.?

@tarsius
Copy link
Member

tarsius commented Oct 5, 2019

I still have to create and send certain perks to kickstarter backers. Before that I can't really ask for more money. Doing that is getting really urgent and I don't want to tackle anything that takes a while even if it were to bring in some money.

@Fuuzetsu
Copy link

Fuuzetsu commented Oct 6, 2019

Understood, thank you.

@TonCherAmi
Copy link

@tarsius could you give us the current status on this? Is it still a low priority thing?

@tarsius
Copy link
Member

tarsius commented Dec 22, 2019

It's a "it's gonna be a lot of work and I don't yet have the time for that" thing.

@TonCherAmi
Copy link

Got it, thanks. Would be awesome to see it at some point.

@Thaodan
Copy link

Thaodan commented Jul 29, 2021

Are PR approves also part of that issue?

@iyefrat
Copy link

iyefrat commented Dec 23, 2021

Substantial work is being done on this front with code-review. It's built on top of magit-section and transient, and has the explicit goal of supporting multiple forges. See here for the current supported feature list per forge. Perhaps this feature can just exist in a separate package, @wandersoncferreira seems to be aiming for basically all the features people want out of this (thanks for the great work man!)

@appetrosyan
Copy link

To anyone who stumbled across this like I did, the package is not currently installable and hasn't been updated since 2021.

@tarsius
Copy link
Member

tarsius commented Feb 1, 2024

I've read that wandersoncferreira/code-review#246 makes that package usable again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

10 participants