ansible-language-server icon indicating copy to clipboard operation
ansible-language-server copied to clipboard

Refactor utils/misc.ts into smaller and more informative utilities

Open priyamsahoo opened this issue 3 years ago • 0 comments

Per https://github.com/ansible/ansible-language-server/pull/5#discussion_r708192985, I don't think it belongs in misc. I'd rather use something distinct, like osUtils, or sysUtils, or pathUtils, or execUtils. But misc? What does it even mean? I've seen linters forbidding pointless package/module names in the past (like utils, or helpers, or functions, or stuff — it's all the same). I feel like this is exactly the same case of abusing the ability to dump things you can't categorize in a huge unmanageable module. Please, think of the high-level purpose of this function and put it into a properly named namespace.

Originally posted by @webknjaz in https://github.com/ansible/ansible-language-server/pull/5#r708196915

priyamsahoo avatar Sep 30 '21 09:09 priyamsahoo