Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jun 3, 2025

This PR moves the PlantUML diagrams from the root /diagrams folder to /assets/diagrams to reduce clutter and improve repository organization.

Changes Made

  • Moved entire /diagrams folder to /assets/diagrams including:

    • README.md with PlantUML setup instructions
    • src/ directory containing .puml source files
    • out/ directory containing generated SVG diagrams
  • Updated VS Code settings in .vscode/settings.json:

    • plantuml.diagramsRoot: diagrams/srcassets/diagrams/src
    • plantuml.exportOutDir: diagrams/outassets/diagrams/out
  • Updated example configuration in the diagrams README.md to reflect the new paths

Verification

  • All PlantUML source files (.puml) and generated outputs (.svg) remain intact
  • VS Code PlantUML extension configuration updated to work with new paths
  • No other code references to old paths exist in the codebase
  • All existing tests continue to pass (108/108 tests passing)

The diagrams functionality remains fully operational with the new folder structure.

Fixes #28.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@github-actions
Copy link

github-actions bot commented Jun 3, 2025

Python 3.13 Test Results

108 tests  ±0   108 ✅ ±0   1s ⏱️ -1s
  1 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit f631dcf. ± Comparison against base commit 063f570.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Jun 3, 2025

Python 3.12 Test Results

108 tests  ±0   108 ✅ ±0   1s ⏱️ -1s
  1 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit f631dcf. ± Comparison against base commit 063f570.

♻️ This comment has been updated with latest results.

Co-authored-by: simonkurtz-MSFT <84809797+simonkurtz-MSFT@users.noreply.github.com>
Copilot AI changed the title [WIP] [Task]: Move PlantUML diagrams to the Assets folder Move PlantUML diagrams to assets folder for better organization Jun 3, 2025
Copilot AI requested a review from simonkurtz-MSFT June 3, 2025 10:52
@simonkurtz-MSFT simonkurtz-MSFT marked this pull request as ready for review June 3, 2025 13:53
@simonkurtz-MSFT simonkurtz-MSFT merged commit 4967f2c into main Jun 5, 2025
6 of 7 checks passed
@simonkurtz-MSFT simonkurtz-MSFT deleted the copilot/fix-28 branch June 5, 2025 11:17
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.

[Task]: Move PlantUML diagrams to the Assets folder

2 participants