"description":"is this PR as minimal and focused as possible, with all code changes centered around a single coherent theme, described in the PR description and title ?"Makesuretoexplainyouranswer"
"description":"important suggestions for the contributors and maintainers of this PR, may include overall structure, primary purpose and best practices. consider using specific filenames, classes and functions names. explain yourself!"
},
"Code suggestions":{
"type":"array",
"maxItems":{{num_code_suggestions}},
"uniqueItems":true,
"items":{
"suggestion number":{
"type":"int",
"description":"suggestion number, starting from 1"
},
"relevant file":{
"type":"string",
"description":"the relevant file name"
},
"suggestion content":{
"type":"string",
{%-ifextended_code_suggestions%}
"description":"aconcretesuggestionformeaningfullyimprovingthenewPRcode.Don't repeat previous suggestions. Add tags with importance measure that matches each suggestion ('important' or 'medium').Donotmakesuggestionsforupdatingoraddingdocstrings,renamingPRtitleanddescription,orlinterlike.
"description":"shortly explain why this suggestion is important"
},
"code example":{
"type":"object",
"properties":{
"before code":{
"type":"string",
"description":"Short and concise code snippet, to illustrate the existing code"
},
"after code":{
"type":"string",
"description":"Short and concise code snippet, to illustrate the improved code"
}
}
}
{%-endif%}
}
},
{%-ifrequire_security%}
"Security concerns":{
"type":"string",
"description":"yes\\no question: does this PR code introduce possible security concerns or issues, like SQL injection, XSS, CSRF, and others ? explain your answer"