Skip to content

Conversation

@svazquezco
Copy link
Collaborator

@svazquezco svazquezco commented Jan 2, 2026

Summary by CodeRabbit

  • Chores

    • Added a project configuration to manage review workflows, auto-review and pre-merge checks, tooling toggles, path/branch filters, and chat/knowledge integration preferences.
  • Documentation

    • Added coding conventions and Python guidelines covering naming, docstrings, linting, testing practices, and dependency/version guidance.

✏️ Tip: You can customize this high-level summary in your review settings.

@svazquezco svazquezco requested a review from a team as a code owner January 2, 2026 12:47
@svazquezco svazquezco requested review from albertsola and d3rky and removed request for a team January 2, 2026 12:47
@coderabbitai
Copy link

coderabbitai bot commented Jan 2, 2026

📝 Walkthrough

Walkthrough

Adds two new repository files: .coderabbit.yaml (CodeRabbit review/CI/tooling and project settings) and .github/.instructions.md (Python coding conventions, linting, and testing guidelines). No code signatures or public APIs were modified.

Changes

Cohort / File(s) Summary
CodeRabbit configuration
/.coderabbit.yaml
New CodeRabbit config defining review workflows, auto-review behavior, pre-merge checks, tool enablement (linters, static analysis), chat integration settings, path filters, and project-specific preferences (Jira keys, repo/branch defaults).
Repository guidance
/.github/.instructions.md
New Markdown document specifying Python coding conventions, environment/config rules, version specifiers, naming and docstring standards, linting/testing guidelines (pytest, AAA, fixtures, mocking), and reliance on pyproject.toml for lint rules.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Pre-merge checks

✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title includes the required JIRA ticket number (MPT-16657) and clearly describes the changes: adding instructions and coderabbit YAML configuration.

📜 Recent review details

Configuration used: Repository YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 4066c64 and e82a172.

📒 Files selected for processing (1)
  • .github/.instructions.md
🚧 Files skipped from review as they are similar to previous changes (1)
  • .github/.instructions.md

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link

github-actions bot commented Jan 2, 2026

Warnings
⚠️

This PR contains 2 commits.

Please squash them into a single commit to keep the git history clean and easy to follow.

Multiple commits are acceptable only in the following cases:

  1. One commit is a technical refactoring, and another introduces business logic changes.
  2. You are doing a complex multi-step refactoring (although in this case we still recommend splitting it into separate PRs).
⚠️

This PR modifies code (2 file(s)) but does not include any changes in the tests/ folder.

Please consider adding or updating tests to cover your changes.

✅ Found Jira issue key in the title: MPT-16657

Generated by 🚫 dangerJS against e82a172

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 3

📜 Review details

Configuration used: Repository YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 01a8f22 and e028be6.

📒 Files selected for processing (2)
  • .coderabbit.yaml
  • .github/.instructions.md
🧰 Additional context used
🪛 markdownlint-cli2 (0.18.1)
.github/.instructions.md

32-32: Bare URL used

(MD034, no-bare-urls)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: build
🔇 Additional comments (3)
.coderabbit.yaml (3)

45-46: Verify the base_branches configuration aligns with your branching strategy.

The configuration enables auto-review for release/* branches in addition to the default branch. Ensure this matches your intended workflow, as this will trigger automatic reviews on all release branches.


173-176: Verify that the code guidelines file pattern is correct.

The configuration references .github/.instructions.md for code guidelines. Confirm that this file path matches the actual location of the instructions file being added in this PR.

This correctly links the CodeRabbit configuration to the coding conventions document, enabling enforcement during reviews.


181-184: LGTM! Jira integration properly configured.

The Jira knowledge base integration is correctly configured with the MPT project key, which aligns with the title requirement on line 59 expecting the MPT- prefix.

@svazquezco svazquezco force-pushed the MPT-16657-add-coderabitt branch from e028be6 to 4066c64 Compare January 2, 2026 13:34
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 2, 2026

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants