GP Saggese

Results 198 comments of GP Saggese

The PR is https://github.com/causify-ai/helpers/pull/754 The next step is to do a PP on Thur to look at the code in action and then merge and deploy

Remaining problems: 1) Once the hook is injected, all the processes collect and generate coverage - We don't have a solution yet - We could use an invoke target that...

1) The email is: https://docs.google.com/document/d/1eZikNlvqrbOjmrukNtlaCkHyhh-hBAJqpNAURAucKW8/edit?tab=t.0 2) We should create a Gdoc in https://drive.google.com/drive/folders/1u53FGm3X5zcAcmAYUAN3vByvYuvoNgv- where everybody can publish the emails and we can keep a log

@heanhsok there is flag to skip the hooks git commit -am "Enable copilot review by default" --no-verify Let's add this in the docs, when we release

We can do this after we are done with https://github.com/causify-ai/helpers/issues/725 The goal is to tell the team "there is a new feature which does XYZ and this is how you...

Good analysis. 1) I don't like the current amp / app mix up, it should be separated (at least in terms of code) 2) I am ok with having a...

Some code and ideas https://github.com/causify-ai/helpers/pull/761

Some notes from this AM discussion - The goal is to create a script ai_reviewer.py - Automate the usual complaints - People run the script, fix the code by hand...

Merged the script from @sonniki Next steps are - Merge my code on top of it - inject_todo.py - Split the md files into chunks for different "targets" (e.g., linter,...