README update on PR merge - Githubissues

ubiquity-os-marketplace / command-ask

0 stars 4 forks source link

README update on PR merge #22

Open Keyrxng opened 2 weeks ago

Keyrxng commented 2 weeks ago

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 commented 2 weeks ago

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)