docs: introduce code review guidelines

[tarasmadan]

Updates #6891

[dvyukov]

Have you reviewed the guidelines manually?

[tarasmadan]

@gemini-cli remove the TIP section at the end

[syzbot-noreply]

🤖 Hi @tarasmadan, I've received your request, and I'm working on it now! You can track my progress in the logs for more details.

[syzbot-noreply]

🤖 AI Assistant: Plan of Action

I have analyzed the request and propose the following plan. This plan will not be executed until it is approved by a maintainer.

Resource Estimate:

• Estimated Tool Calls: ~2
• Files to Modify: 1

Proposed Steps:

• Step 1: Remove the > [!TIP] section at the end of skills/DVYUKOV.md.

Please review this plan. To approve, comment @gemini-cli /approve on this issue. To make changes, comment changes needed.

[tarasmadan]

Have you reviewed the guidelines manually?

Yes. The first version was too long and we compressed it with @ramosian-glider.
Last lines are coming from #7019 and the border between these two PRs was quite blurred for me. Thanks for highlighting.

[tarasmadan]

@gemini-cli /approve

[syzbot-noreply]

🤖 Hi @tarasmadan, I've received your request, and I'm working on it now! You can track my progress in the logs for more details.

[syzbot-noreply]

🤖 I'm sorry @tarasmadan, but I was unable to process your request. Please see the logs for more details.

[tarasmadan]

ptal

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 50.6%. Comparing base (15d8aa1) to head (fc64fac).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

see 6 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.