✨ add automatic projet desactivation and retag#19
Conversation
TeddyAndrieux
left a comment
TeddyAndrieux
left a comment
There was a problem hiding this comment.
I did not reviewed the whole content just few comments otherwise I'm ok if it works:
- Current pre-merge seems to fail
- The new tests you added in this PR does not seems to run in pre-merge
There was a problem hiding this comment.
I'm not sure it make a lot of sense to have it in the repo, or maybe it should be changed in some DESIGN.md or something like that, that state what we are doing and why but a plan does not make much sense IMHO
There was a problem hiding this comment.
Yeah it was an experiment, based on https://code.visualstudio.com/docs/agent-customization/prompt-files
Since we deploy AI review, it may be the correct way to give context to the LLM in charge of the review?
There was a problem hiding this comment.
I'm not sure it's a good idea to me having a DESIGN.md or so, that would contains the reason for each decision would be better than having the "prompt file" used to do this feature, especially if at some point we rework the part you did with a prompt using another one it might be hard for the AI to pick the right one (it may even pick older "prompt file" and do a totally wrong review)
I don't know what is the good approach for it
2 participants
No description provided.