Skip to content

Revert "Package as folder"#18

Merged
ryanlua merged 2 commits intomainfrom
revert-4-package-folder
Dec 18, 2025
Merged

Revert "Package as folder"#18
ryanlua merged 2 commits intomainfrom
revert-4-package-folder

Conversation

@ryanlua
Copy link
Owner

@ryanlua ryanlua commented Dec 18, 2025

Reverts #4

Copilot AI review requested due to automatic review settings December 18, 2025 12:41
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR reverts the "Package as folder" changes from PR #4, restoring the original project structure where the models folder contains an init.luau entry point rather than organizing code within a Purse subfolder.

Key changes:

  • Restructures the models directory to use an init.luau entry point that requires the purse package
  • Moves Packages.project.json from models/Purse/ to models/ directory
  • Updates develop.project.json to reference the models folder directly

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated no comments.

File Description
models/init.luau Adds entry point that requires the purse package from the Packages folder
models/Purse/Packages.project.json Removes the Packages configuration from the Purse subfolder
models/Packages.project.json Adds Packages configuration at the models level, referencing Wally-generated Packages directory and src folder
develop.project.json Updates path from "models/Purse" to "models" to reflect new structure

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Signed-off-by: GitHub <noreply@github.com>
@ryanlua ryanlua merged commit 6cc218e into main Dec 18, 2025
2 checks passed
@ryanlua ryanlua deleted the revert-4-package-folder branch December 18, 2025 23:38
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