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

Ability to update a PR branch based on review comments #27

Closed
neubig opened this issue Aug 29, 2024 · 5 comments
Closed

Ability to update a PR branch based on review comments #27

neubig opened this issue Aug 29, 2024 · 5 comments
Labels
fix-me Try to automatically fix the issue with OpenHands

Comments

@neubig
Copy link
Contributor

neubig commented Aug 29, 2024

Currently this repo has code to send new pull requests based on existing issues. I would like to add the ability to update the pull request based on review comments.

The workflow will be

  • Given the identity of a pull request (based on the numerical id and repo)
  • Download the information from the original issue or issues associated with the PR (based on pattern matching)
  • Download all the review comments
  • For any unsolved comments, make appropriate modifications to the codebase
  • Check if the comments have been successfully resolved

We should update github_resolver/resolve_issues.py so that we have a flag --issue-type that can be set to pr, in which case PRs are considered instead of issues.

Copy link

OpenHands started fixing the issue! You can monitor the progress here.

Copy link

An attempt was made to automatically fix this issue, but it was unsuccessful. A branch named 'openhands-fix-issue-27' has been created with the attempted changes. You can view the branch here. Manual intervention may be required.

@neubig neubig added fix-me Try to automatically fix the issue with OpenHands and removed fix-me Try to automatically fix the issue with OpenHands labels Oct 8, 2024
Copy link

github-actions bot commented Oct 8, 2024

OpenHands started fixing the issue! You can monitor the progress here.

Copy link

github-actions bot commented Oct 8, 2024

An attempt was made to automatically fix this issue, but it was unsuccessful. A branch named 'openhands-fix-issue-27' has been created with the attempted changes. You can view the branch here. Manual intervention may be required.

@neubig
Copy link
Contributor Author

neubig commented Oct 25, 2024

Fixed by #174.

@neubig neubig closed this as completed Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix-me Try to automatically fix the issue with OpenHands
Projects
None yet
1 participant