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.

Sign up for GitHub

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

Jump to bottom

Rename `PackageStore::proto_path()` and `proto_vendor_path()` to something snappier. #135

Open

xfbs opened this issue Oct 20, 2023 · 0 comments

Assignees

Labels

datamodel priority::low type::idea type::refactoring type::style

Milestone

Contributor

xfbs commented Oct 20, 2023

I still think that the "proto_path" is not related to the store, so:

proto/* -> the current package
proto/vendor/* -> the "package store"

So if we would split out the logic for accessing the first path that would be great. I think the weirdness that the vendor dir lives inside of the proto dir creates these blurry borders

xfbs self-assigned this

mara-schulke added datamodel type::style type::refactoring type::idea priority::low labels

mara-schulke added this to Buffrs

mara-schulke added this to the Stabilization milestone

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