Skip to content

Commit af0f713

Browse files
Merge pull request #12 from Raj-G07/patch-1
2 parents b58bccc + 1caf059 commit af0f713

File tree

1 file changed

+107
-0
lines changed

1 file changed

+107
-0
lines changed
Lines changed: 107 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,107 @@
1+
language: 'en'
2+
early_access: false
3+
4+
version: 1
5+
6+
review:
7+
enabled: true
8+
trigger: pull_request
9+
10+
# Files to review
11+
paths:
12+
- "**/*.kt"
13+
- "**/*.kts"
14+
15+
# Excluded files and directories
16+
exclude:
17+
- "**/build/**"
18+
- "**/.gradle/**"
19+
- "**/out/**"
20+
- "**/generated/**"
21+
- "**/node_modules/**"
22+
- "**/*.md"
23+
- "**/*.txt"
24+
25+
# Branch rules
26+
branches:
27+
exclude:
28+
- "dependabot/**"
29+
30+
# Review behavior
31+
behavior:
32+
# Keep feedback concise and actionable
33+
tone: professional
34+
verbosity: balanced
35+
36+
# Avoid nitpicking on trivial formatting
37+
ignore_trivial_changes: true
38+
39+
# Prefer inline suggestions when possible
40+
inline_comments: true
41+
42+
# Do not repeat the same comment multiple times
43+
deduplicate_comments: true
44+
45+
checks:
46+
# Kotlin-specific best practices
47+
kotlin:
48+
enabled: true
49+
50+
# Language correctness
51+
null_safety: true
52+
immutability: true
53+
data_class_usage: true
54+
sealed_class_usage: true
55+
56+
# Code quality
57+
complexity:
58+
enabled: true
59+
max_cyclomatic_complexity: 10
60+
61+
# Coroutines and concurrency
62+
coroutines:
63+
enabled: true
64+
structured_concurrency: true
65+
dispatcher_misuse: true
66+
67+
# Performance considerations
68+
performance:
69+
avoid_unnecessary_allocations: true
70+
collection_usage: true
71+
72+
# API and design
73+
api_design:
74+
visibility_modifiers: true
75+
unnecessary_public_api: true
76+
77+
# Error handling
78+
error_handling:
79+
unchecked_exceptions: true
80+
result_usage: true
81+
82+
# Security checks (lightweight, non-intrusive)
83+
security:
84+
enabled: true
85+
hardcoded_secrets: true
86+
insecure_random: true
87+
unsafe_deserialization: true
88+
89+
# General best practices
90+
general:
91+
dead_code: true
92+
unused_imports: true
93+
naming_conventions: true
94+
documentation_mismatch: true
95+
96+
comments:
97+
# Encourage explanations for non-obvious logic
98+
require_reasoning_for_complex_logic: true
99+
100+
# Avoid commenting on obvious or self-explanatory code
101+
skip_obvious_code: true
102+
103+
summary:
104+
enabled: true
105+
include_strengths: true
106+
include_risks: true
107+
include_recommendations: true

0 commit comments

Comments
 (0)