iljapostnovs / ui5plugin-linter

Linter for UI5 projects
Apache License 2.0
12 stars 3 forks source link

Bug: object naming 'map' → 'Map' or maybe autocomplete with `Object` #137

Open wridgeu opened 1 year ago

wridgeu commented 1 year ago

Describe the bug I think the type should be named with an uppercase M as this is the correct way of spelling it so that JSDoc/TS can recognize it properly and doesn't end up being hinted with an "unresolved" path.

To Reproduce

  1. Create a method that is to be overriden in a subclass within a class that might return an Object e.g. image

  2. then use code completion to generate the implementation of said method image

  3. and you'll get the following output image map doesn't seem right here image

Expected behavior It should create JSDoc with either Map (with an uppercase M) or Object.

Screenshots Done, in previous point.

Desktop (please complete the following information):

Additional context

Project attachment Don't have one at the moment as it is an internal one. Currently don't have much time creating a repro example but will try to make one if really necessary (sorry!).

Appreciate any input! Maybe it isn't even an error and just working as intended or I got the wrong GH repo in the first place. :)