Skip to content

fix(remove-path): prune empty parent directories recursively - #2207

Open
Acuspeedster wants to merge 1 commit into
oscal-compass:developfrom
Acuspeedster:fix/merge-remove-empty-parent-dirs
Open

fix(remove-path): prune empty parent directories recursively#2207
Acuspeedster wants to merge 1 commit into
oscal-compass:developfrom
Acuspeedster:fix/merge-remove-empty-parent-dirs

Conversation

@Acuspeedster

Copy link
Copy Markdown
Contributor

Types of changes

  • Hot fix (emergency fix and release)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Documentation (change which affects the documentation site)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Release (develop -> main)

Quality assurance (all should be covered).

  • My code follows the code style of this project.
  • Documentation for my change is up to date?
  • My PR meets testing requirements.
  • All new and existing tests passed.
  • All commits are signed-off.

Summary

Fix RemovePathAction so it recursively prunes empty parent directories after deleting a file or directory. This prevents stale empty directory shells from lingering after merge or cleanup operations in nested decomposed trees.

Key links:

Before you merge

  • Ensure it is a 'squash commit' if not a release.
  • Ensure CI is currently passing
  • Check sonar. If you are working for a fork a maintainer will reach out, if required.

Signed-off-by: Acuspeedster <arnavrajsingh@gmail.com>
@Acuspeedster
Acuspeedster requested a review from a team as a code owner April 26, 2026 23:07
@Acuspeedster
Acuspeedster force-pushed the fix/merge-remove-empty-parent-dirs branch from c8b581b to a7eee37 Compare April 26, 2026 23:26
@degenaro
degenaro force-pushed the develop branch 2 times, most recently from 64cc98c to 4fa5f11 Compare August 7, 2026 11:15
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