Skip to content

Leverage workspace features and rename crates#33

Open
gibbz00 wants to merge 58 commits into
seffradev:trunkfrom
gibbz00:off_the_rails_v2
Open

Leverage workspace features and rename crates#33
gibbz00 wants to merge 58 commits into
seffradev:trunkfrom
gibbz00:off_the_rails_v2

Conversation

@gibbz00

@gibbz00 gibbz00 commented Sep 14, 2024

Copy link
Copy Markdown
Contributor

Depends on #32

Libraries should avoid locking in logging solutions. Future work could
instead instrument the function behind a feature flag.
@gibbz00 gibbz00 changed the title Leverage workspace features. Leverage workspace features and rename crates Sep 14, 2024
@seffradev

Copy link
Copy Markdown
Owner

Since the "a" in every crate stands for "Asterisk" I don't like prefixing the names with "asterisk-rs". A suggestion is to rename the directories to "rest-interface", "gateway-interface" and "management-interface" (possibly dropping the "interface" part in the directory names) and naming the crates themselves "asterisk-rs-rest-interface" and so on. That makes the names unique, pop up on crates.io when searching for Asterisk and doesn't implicitly contain the word "Asterisk" twice.

@seffradev

Copy link
Copy Markdown
Owner

I don't think making the dependencies at workspace level makes sense for all of them. E.g. AMI and AGI aren't HTTP-based but TCP-based IIRC so e.g. tokio-util is needed for them while request is only for ARI. Before the development of AMI and AGI starts, I think it's best to keep the dependencies local to ARI and move the relevant ones to the workspace level as the other parts get implemented.

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.

2 participants