Skip to content

decrease top-level directories#1837

Merged
shimat merged 4 commits intomainfrom
decrease_dirs
Mar 7, 2026
Merged

decrease top-level directories#1837
shimat merged 4 commits intomainfrom
decrease_dirs

Conversation

@shimat
Copy link
Owner

@shimat shimat commented Mar 7, 2026

Summary by CodeRabbit

  • Chores

    • Reorganized internal project layout and updated CI/build workflow paths.
    • Removed samples git submodule integration.
    • Stopped packaging sample artifacts as part of releases.
    • Added artifact exclusion patterns for build outputs.
  • Documentation

    • Updated docs and build instructions to reflect new tool paths and resource locations.

@shimat shimat self-assigned this Mar 7, 2026
@shimat shimat marked this pull request as ready for review March 7, 2026 05:00
@coderabbitai
Copy link

coderabbitai bot commented Mar 7, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 3160624c-14d1-47ef-909c-9bf40daf9abf

📥 Commits

Reviewing files that changed from the base of the PR and between 7cfea9e and 2699594.

📒 Files selected for processing (2)
  • docfx/docfx.json
  • src/tools/OpenCvSharp.ReleaseMaker/Packer.cs
💤 Files with no reviewable changes (1)
  • src/tools/OpenCvSharp.ReleaseMaker/Packer.cs

📝 Walkthrough

Walkthrough

Repository reorganization moving tooling from tool/ to src/tools/, removal of the samples git submodule, CI/docfx/workflow and docs path updates, and removal of sample-packing logic from ReleaseMaker's Packer.

Changes

Cohort / File(s) Summary
Solution & Projects
OpenCvSharp.sln
Updated solution to reflect projects moved from tool\ to src\tools\, adjusted project GUID mappings and added Solution Items entry.
Tooling code
src/tools/OpenCvSharp.ReleaseMaker/Packer.cs
Removed sample packaging: deleted MakeSamplePackage and GetSampleDstDirName; Pack now only produces binary packages.
CI/CD workflows
.github/workflows/docfx.yml, .github/workflows/publish_nuget.yml, .github/workflows/windows.yml
Updated execution paths from tool/ to src/tools/, docfx path adjustments (docfx/docs/docfx/), and removed submodules: true from checkout in Windows workflow.
Documentation & docfx config
docs/docfx/docfx.json, docs/embedded-builds.md, docfx/docfx.json
Adjusted relative paths and resource globs to align with repo reorganization and moved script/link references to ../src/tools/ or updated docs locations.
Repository metadata
.gitignore, .gitmodules, samples
Added /opencv_artifacts and /tesseract_artifacts to .gitignore; removed samples submodule entry from .gitmodules and deleted the recorded submodule commit line in samples.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

🚥 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 'decrease top-level directories' directly reflects the main objective of the PR, which is reorganizing and consolidating directory structures by moving tool-related projects into src/tools and documentation into docs/docfx.
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 docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch decrease_dirs

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

@shimat shimat merged commit cc75bb9 into main Mar 7, 2026
9 checks passed
@shimat shimat deleted the decrease_dirs branch March 7, 2026 05:56
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