nginxinc / ngx-rust

Rust binding for NGINX

Apache License 2.0

719 stars 59 forks source link

Publish this as `nginx` in crates.io? #17

Closed crlf0710 closed 5 months ago

crlf0710 commented 6 years ago

It seems rather strange from a Rust user's view that a library has the name "rust" in it. Maybe just call this crate nginx or something.

lambda-fairy commented 6 years ago

Yeah, this is mentioned in the official API guidelines:

Crate names should not use -rs or -rust as a suffix or prefix. Every crate is Rust! It serves no purpose to remind users of this constantly.

repi commented 6 years ago

Would indeed be nice to have it published as nginx, no need for a rust prefix/suffix in the crate name as that is exclusively in the Ruts ecosystem. Great to have it in the repository name here though

sehz commented 6 years ago

Thanks for the feedback. Didn't know much about rust guidelines when selecting the name. The current version is more of proof of concept. Will change the crate name in the next release.

yaa110 commented 5 years ago

19 it's been published

ivanitskiy commented 1 year ago

opened https://github.com/nginxinc/ngx-rust/pull/27

dekobon commented 5 months ago

A crate has been published. Closing issues.