I am working on implementing a git diff code explain using OpenAI, marking PR as draft while that's in progress!

The git diff is no bueno!! Script has git diff related functions, will get deleted EOD today (after showing Nik at Community Catchup)

I implemented functionality to use the github-actions summary bot! It's looking pretty good so far. The main plus is that it works!!

Here's a sneak peek of what it can do:

If you want to try this yourself, run the first code block (all the imports), then run the second code cell under "Try automated GitHub PR summary (or cmd+f: RUN FIRST CODE BLOCK (IMPORTS), THEN THIS ONE TO TRY :D DON'T NEED TO RUN ANY OTHER BLOCKS.)

If you want to try other PRs, you can replace the repo_name and pr_number parameters to a different one that has a bot comment already. Right now, it's set to validmind/frontend and 888.

For now, I'm going to keep fiddling with the prompt and see if there's a way to combine/compare this summary with the old one to preserve images and links.

@nrichers I'm not sure how to resolve these, my attempts did not work! Seems like it's still checking against the OLD filename with the uppercasing instead of honouring the new filename without the uppercasing even when I rename everything ughhh 💀

@nrichers I'm not sure how to resolve these, my attempts did not work! Seems like it's still checking against the OLD filename with the uppercasing instead of honouring the new filename without the uppercasing even when I rename everything ughhh 💀

@validbeck this branch had a legitimate merge conflict with one of the notebooks (the source, not the output). Fixed that, added all the files, and continued the merge, followed by render. Looks fine now, previously problematic paths included.

Note that the PR explanation step is failing due to excessive length, going to merge as not relevant to the technical changes:

openai.BadRequestError: Error code: 400 - {'error': {'message': "Invalid 'messages[0].content': string too long. Expected a string with maximum length 1048576, but got a string with length 4988725 instead.", 'type': 'invalid_request_error', 'param': 'messages[0].content', 'code': 'string_above_max_length'}}

this branch had a legitimate merge conflict with one of the notebooks (the source, not the output). Fixed that, added all the files, and continued the merge, followed by render. Looks fine now, previously problematic paths included.

Doesn't make get-source override the existing notebooks? Bit confused as to how the source had a conflict 🤔

Doesn't make get-source override the existing notebooks? Bit confused as to how the source had a conflict 🤔

Yes, it does, and one of those source notebooks had a merge conflict. It's like any other source file that has a merge conflict, you need to actually resolve it. Kinda rare for content we merge in from other repos, but it can happen.