Skip to content

Conversation

@Rob-Hague
Copy link

@Rob-Hague Rob-Hague commented Feb 7, 2026

None of these properties are necessary (most of it looks auto-generated from an IDE)

Summary by CodeRabbit

  • Chores
    • Simplified build configurations across projects by removing obsolete settings and clearing references to generated artifacts.

None of these properties are necessary (most of it looks auto-generated from an IDE)
@coderabbitai
Copy link

coderabbitai bot commented Feb 7, 2026

📝 Walkthrough

Walkthrough

These changes remove various build configuration properties and artifact management declarations from multiple .NET project files. The modifications include eliminating GenerateTargetFrameworkAttribute settings, package generation controls, FastEndpoints dependency reference, and cleanup specifications for build-generated outputs across four projects.

Changes

Cohort / File(s) Summary
Package Generation Configuration
src/example-auth-tls/example-auth-tls.csproj, src/example-basic/example-basic.csproj
Removed GeneratePackageOnBuild property, allowing projects to rely on default packaging behavior instead of explicitly disabling package generation on build.
Build Artifacts and Target Framework Cleanup
src/dummy-http-server/dummy-http-server.csproj, src/net-questdb-client-benchmarks/net-questdb-client-benchmarks.csproj
Removed GenerateTargetFrameworkAttribute declarations and eliminated multiple ItemGroup blocks that managed obj/** and generated artifact cleanup (Compile, Content, EmbeddedResource, and None entries). Also removed FastEndpoints package reference from dummy-http-server.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 Artifacts trimmed, configs cleaned with care,
Build files lightened, no clutter to bear,
FastEndpoints gone, frameworks set free,
Simpler projects now—tidy and spree!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Remove crud from csproj files' accurately describes the main change: removing unnecessary properties and artifacts from multiple .csproj files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

No actionable comments were generated in the recent review. 🎉


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.

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