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

Is it possible to approve a pull request from within forge? #377

Open
BigTheta opened this issue Jul 7, 2021 · 10 comments
Open

Is it possible to approve a pull request from within forge? #377

BigTheta opened this issue Jul 7, 2021 · 10 comments
Labels
area: actions enhancement New feature or request

Comments

@BigTheta
Copy link

BigTheta commented Jul 7, 2021

I have checked the manual, FAQ and reddit, I don't see anywhere where its even been asked before.

I was asked to review a pull request, I was able to view the commits and comment on the PR. Now, I would like to approve the PR. I know gh has this functionality. It seems like the sort of functionality forge was intended to support?

@tarsius
Copy link
Member

tarsius commented Jul 7, 2021

Not yet, I believe. Supporting it isn't a top priority but eventually it will probably be implemented. (The projects I am involved in for the most part work by merging or not merging pull-requests without explicitly and beforehand approve or reject it.)

@tarsius tarsius added the enhancement New feature or request label Jul 7, 2021
@jpneverwas

This comment was marked as off-topic.

@tarsius
Copy link
Member

tarsius commented Jul 16, 2021

Is it possible to join/approve a pull request from within forge?

I just remembered that I implemented this quite recently in 3112ade. It's available from the merge menu, but not by default so...

@tarsius
Copy link
Member

tarsius commented Jul 16, 2021

Certain transient popups hide some of their suffix commands by default. These additional suffixes can be enabled interactively. TL;DR Enter the transient as usual and type C-x l.

This is a canned response; it may not apply 100% in this case.

@tarsius

This comment was marked as off-topic.

@tarsius
Copy link
Member

tarsius commented Jul 16, 2021

I have checked the manual,

I've just added it to the manual.

@tarsius tarsius closed this as completed Jul 16, 2021
@BigTheta
Copy link
Author

Thank you. Much appreciated.

@BigTheta
Copy link
Author

BigTheta commented Sep 1, 2021

Sorry, I never actually attempted this until just now.

I just remembered that I implemented this quite recently in [3112ade]

To be clear, you are taking about merging (not approving?) I don't see anything in that commit regarding approving reviews.

It's available from the merge menu, but not by default so...

I'm looking at the merge menu with suffixes enabled and I still don't see anything regarding approving the topic?

image

@tarsius
Copy link
Member

tarsius commented Sep 1, 2021

To be clear, you are taking about merging (not approving?)

Yeah. Your use of the word "join" confused me. That sounded like a synonym for "merge".

So no. Approving still isn't supported and adding support is a low priority.

@tarsius tarsius reopened this Sep 1, 2021
@tarsius tarsius changed the title Is it possible to join/approve a pull request from within forge? Is it possible to approve a pull request from within forge? Apr 6, 2022
@aiguofer
Copy link

Check out https://github.com/wandersoncferreira/code-review (I just found it looking for better ways to do reviews using forge)

Slightly weird switching between forge and code-review, but they do integrate fairly well.

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

No branches or pull requests

4 participants