search

microsoft / devicescript

TypeScript for Tiny IoT Devices (ESP32, RP2040, ...)
https://microsoft.github.io/devicescript/
MIT License
3.27k stars 118 forks source link

rename devsdbg to devicescriptdebugger #81

Closed pelikhan closed 1 year ago

pelikhan commented 1 year ago

we use full "devicescript" in other service names

image
mmoskal commented 1 year ago

I think you should use full name not camel name in tree. I want to rename camel devicescriptManager to devsmgr. These super long names get into constants in C which then don't fit on screen.

pelikhan commented 1 year ago

Right…. But abbreviation are harder to understand. Maybe another flag to control C constant generation?

From: Michał Moskal @.> Sent: Wednesday, February 1, 2023 6:28:00 AM To: microsoft/devicescript @.> Cc: Peli de Halleux @.>; Author @.> Subject: Re: [microsoft/devicescript] rename devsdbg to devicescriptdebugger (Issue #81)

I think you should use full name not camel name in tree. I want to rename camel devicescriptManager to devsmgr. These super long names get into constants in C which then don't fit on screen.

— Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fdevicescript%2Fissues%2F81%23issuecomment-1412151074&data=05%7C01%7Cjhalleux%40microsoft.com%7Cb8e7a356f31c45dc38f508db046085bb%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C638108584835723911%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=ghPWgBX4x9smon2v7Ge%2BwaaAoFyugHXgBgd6iKWjO3E%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAA73QKPE4FXO72R2TDKKTXTWVJXHBANCNFSM6AAAAAAUMAVY7A&data=05%7C01%7Cjhalleux%40microsoft.com%7Cb8e7a356f31c45dc38f508db046085bb%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C638108584835882402%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=SKbNmO4kXrsQYA5CS6oCzJUuWuG%2BF6T7xd6vlFiZNHQ%3D&reserved=0. You are receiving this because you authored the thread.Message ID: @.***>

mmoskal commented 1 year ago

Then don't use abbreviations in the tree - just use the full service name. The lengthy constants also applies to TS...