Skip to content

chore: update formatter configuration to include new project locals - #29

Merged
yordis merged 1 commit into
mainfrom
yordis/formatter
Nov 18, 2025
Merged

yordis merged 1 commit into
mainfrom
yordis/formatter

Conversation

@yordis

@yordis yordis commented Nov 18, 2025 •

Copy link
Copy Markdown
Member

Note

Add project: 2 and project: 3 to locals_without_parens in .formatter.exs.

Written by Cursor Bugbot for commit 4f9c8eb. This will update automatically on new commits. Configure here.

@coderabbitai

coderabbitai Bot commented Nov 18, 2025 •

Copy link
Copy Markdown

Walkthrough

The .formatter.exs configuration file was updated to add two new entries (project: 2 and project: 3) to the locals_without_parens option, extending the formatter's configuration for handling local functions without parentheses.

Changes

Cohort / File(s) Summary
Formatter Configuration
\.formatter\.exs
Added project: 2 and project: 3 to the locals_without_parens configuration

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A formatter's heart, it beats so true,
With project: 2 and project: 3 too,
No parens needed, cleaner the sight,
Our code hops along, formatted just right! 🐰✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly describes the main change: updating formatter configuration to add new project locals, which matches the changeset exactly.
Description check ✅ Passed The description is related to the changeset, specifying the addition of project/2 and project/3 to locals_without_parens in .formatter.exs.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch yordis/formatter

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between df29a55 and 4f9c8eb.

📒 Files selected for processing (1)
  • .formatter.exs (1 hunks)
⏰ 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: Quality Assurance (1.18.x, 27)
🔇 Additional comments (1)
.formatter.exs (1)

7-8: LGTM!

The formatter configuration update correctly adds project: 2 and project: 3 to the locals_without_parens list. Both entries follow the established pattern, are properly ordered alphabetically within the list, and are correctly exported on line 19.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
@yordis
yordis marked this pull request as ready for review November 18, 2025 04:07
@yordis
yordis merged commit 0cfdccd into main Nov 18, 2025
6 of 9 checks passed
@yordis
yordis deleted the yordis/formatter branch November 18, 2025 04:21
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.

1 participant