Skip to content

fix(merge): clean up distributed subdir after merge - #2193

Merged
degenaro merged 2 commits into
oscal-compass:developfrom
Acuspeedster:fix/merge-cleanup-distributed-subdirs
Aug 28, 2026
Merged

fix(merge): clean up distributed subdir after merge#2193
degenaro merged 2 commits into
oscal-compass:developfrom
Acuspeedster:fix/merge-cleanup-distributed-subdirs

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 merge cleanup so decomposed subdirectories are removed when their parent model element is merged back into a top-level file. This prevents stale distributed content from being left behind after merge operations such as catalog.metadata.

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.

@Acuspeedster
Acuspeedster requested a review from a team as a code owner April 8, 2026 21:25
Signed-off-by: Acuspeedster <arnavrajsingh@gmail.com>
@Acuspeedster
Acuspeedster force-pushed the fix/merge-cleanup-distributed-subdirs branch from 7537349 to aef1121 Compare April 8, 2026 21:37
@degenaro
degenaro force-pushed the develop branch 2 times, most recently from 64cc98c to 4fa5f11 Compare August 7, 2026 11:15

@degenaro degenaro left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@degenaro
degenaro merged commit 3458fcf into oscal-compass:develop Aug 28, 2026
16 checks passed
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.

2 participants