Skip to content

Commit 1a31185

Browse files
authored
Merge pull request #8 from masterpointio/feature/git-configs
feat: adds GH + CRabbit configs
2 parents 235c45e + 721721a commit 1a31185

File tree

4 files changed

+116
-0
lines changed

4 files changed

+116
-0
lines changed

.coderabbit.yaml

Lines changed: 90 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,90 @@
1+
# Docs: https://docs.coderabbit.ai/configure-coderabbit
2+
# Schema: https://coderabbit.ai/integrations/schema.v2.json
3+
# Support: https://discord.gg/GsXnASn26c
4+
5+
language: en
6+
7+
tone_instructions: |
8+
Provide feedback in a professional, friendly, constructive, and concise tone.
9+
Offer clear, specific suggestions and best practices to help enhance the code quality and promote learning.
10+
11+
early_access: true
12+
13+
knowledge_base:
14+
# The scope of learnings to use for the knowledge base.
15+
# `local` uses the repository's learnings,
16+
# `global` uses the organization's learnings,
17+
# `auto` uses repository's learnings for public repositories and organization's learnings for private repositories.
18+
# Default value: `auto`
19+
learnings:
20+
scope: global
21+
issues:
22+
scope: global
23+
pull_requests:
24+
scope: global
25+
26+
reviews:
27+
profile: chill
28+
auto_review:
29+
# Ignore reviewing if the title of the pull request contains any of these keywords (case-insensitive)
30+
ignore_title_keywords:
31+
- wip
32+
- draft
33+
- test
34+
# Set the commit status to 'pending' when the review is in progress and 'success' when it is complete.
35+
commit_status: false
36+
# Post review details on each review. Additionally, post a review status when a review is skipped in certain cases.
37+
review_status: false
38+
path_instructions:
39+
- path: "**/*.tf"
40+
instructions: |
41+
You're a Terraform expert who has thoroughly studied all the documentation from Hashicorp https://developer.hashicorp.com/terraform/docs and OpenTofu https://opentofu.org/docs/.
42+
You have a strong grasp of Terraform syntax and prioritize providing accurate and insightful code suggestions.
43+
As a fan of the Cloud Posse / SweetOps ecosystem, you incorporate many of their best practices https://docs.cloudposse.com/best-practices/terraform/ while balancing them with general Terraform guidelines.
44+
tools:
45+
# By default, all tools are enabled.
46+
# Masterpoint uses Trunk (https://trunk.io) so we do not need a lot of this feedback due to overlap.
47+
shellcheck:
48+
enabled: false
49+
ruff:
50+
enabled: false
51+
markdownlint:
52+
enabled: false
53+
github-checks:
54+
enabled: false
55+
languagetool:
56+
enabled: false
57+
biome:
58+
enabled: false
59+
hadolint:
60+
enabled: false
61+
swiftlint:
62+
enabled: false
63+
phpstan:
64+
enabled: false
65+
golangci-lint:
66+
enabled: false
67+
yamllint:
68+
enabled: false
69+
gitleaks:
70+
enabled: false
71+
checkov:
72+
enabled: false
73+
detekt:
74+
enabled: false
75+
eslint:
76+
enabled: false
77+
rubocop:
78+
enabled: false
79+
buf:
80+
enabled: false
81+
regal:
82+
enabled: false
83+
actionlint:
84+
enabled: false
85+
pmd:
86+
enabled: false
87+
cppcheck:
88+
enabled: false
89+
circleci:
90+
enabled: false

.github/CODEOWNERS

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
# Use this file to define individuals or teams that are responsible for code in a repository.
2+
# Read more: <https://help.github.com/articles/about-codeowners/>
3+
#
4+
# Order is important: the last matching pattern takes the most precedence
5+
6+
# These owners will be the default owners for everything
7+
* @masterpointio/masterpoint-internal

.github/PULL_REQUEST_TEMPLATE.md

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,15 @@
1+
## what
2+
3+
- Describe high-level what changed as a result of these commits (i.e. in plain-english, what do these changes mean?)
4+
- Use bullet points to be concise and to the point.
5+
6+
## why
7+
8+
- Provide the justifications for the changes (e.g. business case).
9+
- Describe why these changes were made (e.g. why do these commits fix the problem?)
10+
- Use bullet points to be concise and to the point.
11+
12+
## references
13+
14+
- Link to any supporting GitHub issues or helpful documentation to add some context (e.g. Stackoverflow).
15+
- Use `closes #123`, if this PR closes a GitHub issue `#123`

.trunk/configs/.markdownlint.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,3 +8,7 @@ line_length: false
88
spaces: false
99
url: false
1010
whitespace: false
11+
12+
# Ignore MD041/first-line-heading/first-line-h1
13+
# Error: First line in a file should be a top-level heading
14+
MD041: false

0 commit comments

Comments
 (0)