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

README update on PR merge #22

Open
Keyrxng opened this issue Oct 27, 2024 · 1 comment
Open

README update on PR merge #22

Keyrxng opened this issue Oct 27, 2024 · 1 comment

Comments

@Keyrxng
Copy link
Member

Keyrxng commented Oct 27, 2024

This might deserve it's own plugin but it could easily be an extension of this one through #11.

  • When a PR is accepted and merged the README should be updated to reflect the changes
  • context window? task and pr body and convos + review comments + PR diff?
  • should it be automated or should it be invoked manually before/after merge on the PR (to handle PRs that should be skipped for this sort of treatment like merging development into main, new package releases etc)?
  • Some control features are needed in my opinion

original context

@0x4007
Copy link
Member

0x4007 commented Oct 28, 2024

I think for the first version it could be useful to build it in here because it can use the same code that was recently produced to parse the diff and add to context etc.

It's a nice approach that attempts to get as much relevant context as possible without overloading the max tokens per request.

I think in a final version it should accommodate commits straight to development (non pull requests) but this may be more tricky to get right as there will be very little relevant context (no issue specification etc)

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

No branches or pull requests

2 participants