Skip to content

Restructure into src/include/data/docs layout and fix filename typo#1

Merged
Denarzai merged 1 commit into
mainfrom
restructure/src-include-layout
Jul 4, 2026
Merged

Restructure into src/include/data/docs layout and fix filename typo#1
Denarzai merged 1 commit into
mainfrom
restructure/src-include-layout

Conversation

@Denarzai

@Denarzai Denarzai commented Jul 4, 2026

Copy link
Copy Markdown
Owner

What this changes

  • Moves all .cpp files to src/, headers to include/, sample data files to data/, and UML artifacts (finalUML.pdf, projuml.drawio, class-diagram.png) to docs/ — the repo root previously held 46 files
  • Renames perfomanceReview.cpp/.hperformanceReview.cpp/.h (filename typo) and updates the three #include lines that referenced it; the class name was already spelled correctly, so no code identifiers change
  • Updates README build instructions for the new layout (src/*.cpp -Iinclude) for both g++ and MSVC, documents the data/ sample-data flow, and adds a repository structure table
  • Adds a CI workflow that compiles the project with g++ on every push, plus a build badge

Related issue

None — from the profile audit (repo organization).

Checklist

  • The project still builds (instructions in the README) — verified by the CI run on this PR
  • The change is focused on one thing
  • Documentation updated if behaviour changed

🤖 Generated with Claude Code

Move implementation files to src/, headers to include/, sample data to
data/, and UML artifacts to docs/. Rename perfomanceReview.* to
performanceReview.* and update the includes that referenced it. Update
build instructions accordingly and add a CI compile check.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@Denarzai Denarzai merged commit a5d057b into main Jul 4, 2026
1 check passed
@Denarzai Denarzai deleted the restructure/src-include-layout branch July 4, 2026 10:16
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