Skip to content

Conversation

@Nolan-O
Copy link

@Nolan-O Nolan-O commented Jul 10, 2024

Not sure if this is an acceptable feature but I wanted to see if it was possible 🙂

Allows a roblox folder's contents to be the sum of multiple flesystem folders' contents. Enhances the ability for rojo to change a project's form based on project files. E.G. multi-place games, libraries with tiered features for releases, a tests.project.json file

This feature may conflict with syncback since it introduces an ambiguity where new contents of a roblox folder should be stored. If possible, I think it would be acceptable to use the first path in the list of paths as a "dominant" one, treating it like the base folder on the filesystem.

Notes:

  • $paths is mutually exclusive with $path
  • $paths can only be on Folder nodes (maybe unnecessary?)
  • $paths can only point to folders
  • $paths ignores the metadata files in those folders (because none of them are the sole instance being synced)

Nolan-O added 5 commits July 9, 2024 16:44
Allows a folder's contents to be the sum of multiple folders' contents
Ideal for composing forms of a project depending on rojo's project files
 e.g. multi-place games, libraries with tiered features for releases, etc
@Dekkonot
Copy link
Member

Howdy! I'm going to wait to give this a review until syncback is merged since rather selfishly I don't want to handle the merge conflicts on a PR that's already massive. I will get back to you though, promise.

@Dekkonot Dekkonot added type: enhancement Feature or improvement that should potentially happen scope: cli Relevant to the Rojo CLI labels Jul 15, 2024
@Nolan-O
Copy link
Author

Nolan-O commented Jul 18, 2024

Agreed tbh, makes sense for the large structural change to go first. This PR should have to work with syncback rather than the other way around.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: cli Relevant to the Rojo CLI type: enhancement Feature or improvement that should potentially happen

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants