Skip to content

Conversation

@nfsdns
Copy link
Contributor

@nfsdns nfsdns commented Dec 24, 2025

closes #11921

Summary by CodeRabbit

  • Chores
    • Updated boilerplate template to refine file exclusion logic when offline database mode is disabled.

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

Copilot AI review requested due to automatic review settings December 24, 2025 06:22
@coderabbitai
Copy link

coderabbitai bot commented Dec 24, 2025

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Note

Other AI code review bot(s) detected

CodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review.

Walkthrough

The template.json configuration file in the Boilerplate template is updated to add two new exclusion paths for Todo-related controllers under the condition when offline database is disabled, refining which files are excluded during template generation.

Changes

Cohort / File(s) Summary
Template Configuration
src/Templates/Boilerplate/Bit.Boilerplate/.template.config/template.json
Added two exclusion paths under the offlineDb != true condition: src/Shared/Controllers/Todo/** and src/Server/Boilerplate.Server.Api/Controllers/Todo/** to control which Todo controller files are excluded from template generation

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Two paths excluded with care,
When offline must beware,
Todo controllers tucked away so tight,
The template hops left and right! ✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title accurately describes the main change: excluding Todo controllers when offlineDb is disabled, and references the specific issue #11921.

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.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR attempts to exclude Todo controller directories when the offlineDb feature is disabled by adding two exclusion patterns to the template configuration.

Key Changes:

  • Adds exclusions for src/Shared/Controllers/Todo/** and src/Server/Boilerplate.Server.Api/Controllers/Todo/** in the offlineDb != true condition block

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: 1

📜 Review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Knowledge base: Disabled due to Reviews -> Disable Knowledge Base setting

📥 Commits

Reviewing files that changed from the base of the PR and between c46c9ed and 1599ae9.

📒 Files selected for processing (1)
  • src/Templates/Boilerplate/Bit.Boilerplate/.template.config/template.json

@ysmoradi ysmoradi changed the title Exclude todoControllers when offlineDb disabled mode #11921 Exclude TodoTableController when offlineDb disabled mode and sample is true #11921 Dec 24, 2025
@ysmoradi ysmoradi merged commit a48409c into bitfoundation:develop Dec 24, 2025
3 checks passed
@msynk msynk changed the title Exclude TodoTableController when offlineDb disabled mode and sample is true #11921 Exclude TodoTableController when offlineDb disabled mode and sample is true (#11921) Dec 24, 2025
@nfsdns nfsdns deleted the 11921-source-generator-exclude-todo-controllers branch January 6, 2026 07:00
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.

bit Boilerplate incorrectly includes Todo sync controllers when OfflineDb is false and sample is true

2 participants