Skip to content

chore(config): remove stale assets/ folder references#526

Merged
nanotaboada merged 1 commit intomasterfrom
fix/remove-assets-references
Mar 17, 2026
Merged

chore(config): remove stale assets/ folder references#526
nanotaboada merged 1 commit intomasterfrom
fix/remove-assets-references

Conversation

@nanotaboada
Copy link
Owner

@nanotaboada nanotaboada commented Mar 17, 2026

Summary

  • Removes COPY assets/ ./assets/ from Dockerfile — the folder was
    deleted when the README was migrated to Mermaid diagrams, causing the
    CD pipeline Docker build to fail
  • Removes ^assets/.* exclusion from codecov.yml
  • Removes !**/assets/** exclusion from .coderabbit.yaml

Root Cause

The assets/ folder (containing structure.svg and swagger.png) was
deleted in a prior commit but references to it were left behind in three
configuration files.


This change is Reviewable

Summary by CodeRabbit

  • Chores

    • Removed stale asset folder references from build, Docker, and code quality scanning configurations.
  • Documentation

    • Updated changelog to document cleanup of obsolete asset folder references.

@coderabbitai
Copy link

coderabbitai bot commented Mar 17, 2026

Warning

Ignoring CodeRabbit configuration file changes. For security, only the configuration from the base branch is applied for open source repositories.

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: 35f8ac30-7398-4c0d-8cd6-21e994057181

📥 Commits

Reviewing files that changed from the base of the PR and between fc081b6 and 404b31c.

📒 Files selected for processing (4)
  • .coderabbit.yaml
  • CHANGELOG.md
  • Dockerfile
  • codecov.yml
💤 Files with no reviewable changes (3)
  • Dockerfile
  • .coderabbit.yaml
  • codecov.yml

Walkthrough

The changes remove stale references and exclusion rules for the assets/ folder across configuration and build files following its deletion during README migration to Mermaid diagrams. This includes updates to the Dockerfile build instructions, codecov coverage ignore patterns, code review path filters, and a CHANGELOG entry documenting these removals.

Changes

Cohort / File(s) Summary
Assets folder cleanup
.coderabbit.yaml, codecov.yml, Dockerfile
Removed exclusion/ignore patterns and copy instructions referencing the deleted assets/ folder from review filters, coverage configuration, and container build steps.
Documentation
CHANGELOG.md
Added entry documenting removal of stale assets/ folder references from configuration files following README migration to Mermaid diagrams.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • feature/coverage-settings #328: Modifies asset-related paths and exclusion rules in repository configuration, indicating similar cleanup of assets folder references.
🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title uses the required Conventional Commits format (chore:), is well under 80 characters (53 chars), and clearly describes the main change of removing stale assets/ folder references across config 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
  • Commit unit tests in branch fix/remove-assets-references
  • 🛠️ sync documentation: Commit on current branch
  • 🛠️ sync documentation: Create PR
  • 🛠️ enforce http error handling: Commit on current branch
  • 🛠️ enforce http error handling: Create PR
  • 🛠️ idiomatic review: Commit on current branch
  • 🛠️ idiomatic review: Create PR
  • 🛠️ verify api contract: Commit on current branch
  • 🛠️ verify api contract: Create PR
📝 Coding Plan
  • Generate coding plan for human review comments

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.

@sonarqubecloud
Copy link

@nanotaboada nanotaboada merged commit 64f5cbb into master Mar 17, 2026
9 checks passed
@nanotaboada nanotaboada deleted the fix/remove-assets-references branch March 17, 2026 19:22
@codecov
Copy link

codecov bot commented Mar 17, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (fc081b6) to head (404b31c).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##            master      #526   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            3         3           
  Lines          111       111           
=========================================
  Hits           111       111           
Components Coverage Δ
Services 100.00% <ø> (ø)
Routes 100.00% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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