Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

architecture: Conceptually merge repositories into pallets #340

Open
ethanjli opened this issue Dec 9, 2024 · 0 comments
Open

architecture: Conceptually merge repositories into pallets #340

ethanjli opened this issue Dec 9, 2024 · 0 comments
Milestone

Comments

@ethanjli
Copy link
Member

ethanjli commented Dec 9, 2024

Currently we have two separate but related concepts: Forklift repositories, and Forklift pallets. This is because initially Forklift pallets could only import packages from Forklift repositories. But we've gradually been turning Forklift pallets into a superset of Forklift repositories: now Forklift pallets can deploy locally-defined packages, and they can define packages and they can be required by other pallets as if they were repositories. Currently, it's possible to define and compose pallets without defining repositories at all. So it would be simpler to describe Forklift if we had only the concept of pallets without also having the concept of repositories, since pallets are now basically a superset of the functionality of repositories.

There may be some implications for how we resolve packages and for how we load packages from the cache (since currently pallets are cached on the filesystem in unmerged form, and we load packages from the cache by searching the filesystem) and for how we bundle packages in pallet bundles and for how we load packages from pallet bundles - I haven't thought through those implications yet.

@ethanjli ethanjli added this to the Backlog milestone Dec 9, 2024
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

No branches or pull requests

1 participant