Bug#1035656: ITP: ruby-ruby-openai -- OpenAI API + Ruby! 烙❤️
On 08/05/23 12:37 pm, Sandro Tosi wrote: I just kept the upstream description. It could be removed if that is a problem. the upstream description is quite poor, as the Debian maintainer you should provide a better one; and at the same time remove the emojis, as they provide no value in describing what the package does (which is the sole purpose of the Description field) Just like other bugs, you can file a bug with appropriate severity and even offer a patch. This is uploaded already, and can be considered in the next upload (once accepted). the point of sending ITPs to d-devel is to get them reviewed. this is that review. please fix it now, given 1. it's not been accepted thru NEW yet, 2. you can commit the change right now to the repo, 3. there's no package to report a bug against, since it's not in the archive yet. Fixed in NEW/3.7.0-2. Thanks for the review. I can understand the need to have good descriptions, but I don't think treating them as bugs of grave severity is warranted. OpenPGP_0x8F53E0193B294B75.asc Description: OpenPGP public key OpenPGP_signature Description: OpenPGP digital signature
Bug#1035656: ITP: ruby-ruby-openai -- OpenAI API + Ruby! 烙❤️
> >> I just kept the upstream description. It could be removed if that is a > >> problem. > > > > the upstream description is quite poor, as the Debian maintainer you > > should provide a better one; and at the same time remove the emojis, > > as they provide no value in describing what the package does (which is > > the sole purpose of the Description field) > > Just like other bugs, you can file a bug with appropriate severity and > even offer a patch. This is uploaded already, and can be considered in > the next upload (once accepted). the point of sending ITPs to d-devel is to get them reviewed. this is that review. please fix it now, given 1. it's not been accepted thru NEW yet, 2. you can commit the change right now to the repo, 3. there's no package to report a bug against, since it's not in the archive yet. -- Sandro "morph" Tosi My website: http://sandrotosi.me/ Me at Debian: http://wiki.debian.org/SandroTosi Twitter: https://twitter.com/sandrotosi
Bug#1035656: ITP: ruby-ruby-openai -- OpenAI API + Ruby! 烙❤️
On 08/05/23 12:26 pm, Sandro Tosi wrote: On Mon, May 8, 2023 at 2:02 AM Praveen Arimbrathodiyil wrote: On 08/05/23 10:22 am, Sandro Tosi wrote: On Sun, 7 May 2023 14:37:35 +0530 Praveen Arimbrathodiyil wrote: Package: wnpp severity: wishlist Owner: Pirate Praveen Package https://rubygems.org/gems/ruby-openai (License: Expat) why the double ruby- prefix? Because the gem name is ruby-openai. We used to skip double ruby prefix earlier, but we have changed that practice now. this seems like a step in the wrong direction, and it would be fixed. a double prefix provides no benefits, confuses people and it's probably only something the ruby team does (AFAIK); i believe it should be just `ruby-openai` There are cases when we have to differentiate between two gems like this https://rubygems.org/gems/parser https://rubygems.org/gems/ruby_parser node also uses this practice when upstream module include node- in it. are emojis really needed in the package description? I just kept the upstream description. It could be removed if that is a problem. the upstream description is quite poor, as the Debian maintainer you should provide a better one; and at the same time remove the emojis, as they provide no value in describing what the package does (which is the sole purpose of the Description field) Just like other bugs, you can file a bug with appropriate severity and even offer a patch. This is uploaded already, and can be considered in the next upload (once accepted). OpenPGP_0x8F53E0193B294B75.asc Description: OpenPGP public key OpenPGP_signature Description: OpenPGP digital signature
Bug#1035656: ITP: ruby-ruby-openai -- OpenAI API + Ruby! 烙❤️
On Mon, May 8, 2023 at 2:02 AM Praveen Arimbrathodiyil wrote: > > > > On 08/05/23 10:22 am, Sandro Tosi wrote: > > On Sun, 7 May 2023 14:37:35 +0530 Praveen Arimbrathodiyil > > wrote: > >> Package: wnpp > >> severity: wishlist > >> Owner: Pirate Praveen > >> > >> Package https://rubygems.org/gems/ruby-openai (License: Expat) > > > > > > why the double ruby- prefix? > > Because the gem name is ruby-openai. We used to skip double ruby prefix > earlier, but we have changed that practice now. this seems like a step in the wrong direction, and it would be fixed. a double prefix provides no benefits, confuses people and it's probably only something the ruby team does (AFAIK); i believe it should be just `ruby-openai` > > > are emojis really needed in the package > > description? > > I just kept the upstream description. It could be removed if that is a > problem. the upstream description is quite poor, as the Debian maintainer you should provide a better one; and at the same time remove the emojis, as they provide no value in describing what the package does (which is the sole purpose of the Description field) -- Sandro "morph" Tosi My website: http://sandrotosi.me/ Me at Debian: http://wiki.debian.org/SandroTosi Twitter: https://twitter.com/sandrotosi
Bug#1035656: ITP: ruby-ruby-openai -- OpenAI API + Ruby! 烙❤️
On 08/05/23 10:22 am, Sandro Tosi wrote: On Sun, 7 May 2023 14:37:35 +0530 Praveen Arimbrathodiyil wrote: Package: wnpp severity: wishlist Owner: Pirate Praveen Package https://rubygems.org/gems/ruby-openai (License: Expat) why the double ruby- prefix? Because the gem name is ruby-openai. We used to skip double ruby prefix earlier, but we have changed that practice now. are emojis really needed in the package description? I just kept the upstream description. It could be removed if that is a problem. OpenPGP_0x8F53E0193B294B75.asc Description: OpenPGP public key OpenPGP_signature Description: OpenPGP digital signature
Bug#1035656: ITP: ruby-ruby-openai -- OpenAI API + Ruby! 烙❤️
On Sun, 7 May 2023 14:37:35 +0530 Praveen Arimbrathodiyil wrote: > Package: wnpp > severity: wishlist > Owner: Pirate Praveen > > Package https://rubygems.org/gems/ruby-openai (License: Expat) why the double ruby- prefix? are emojis really needed in the package description?
Bug#1035656: ITP: ruby-ruby-openai -- OpenAI API + Ruby! 烙❤️
Package: wnpp severity: wishlist Owner: Pirate Praveen Package https://rubygems.org/gems/ruby-openai (License: Expat) OpenPGP_0x8F53E0193B294B75.asc Description: OpenPGP public key OpenPGP_signature Description: OpenPGP digital signature