UAST type coverage report

bblfsh / documentation

Babelfish documentation (GitBook)

https://docs.sourced.tech/babelfish

Creative Commons Attribution Share Alike 4.0 International

41 stars 30 forks source link

UAST type coverage report #253

Closed bzz closed 5 years ago

bzz commented 5 years ago

Initial version of the report that only includes UAST types, found in all driver's fixtures. Triggered by make types.

Algorithm:

discover all drivers, available on-line
clone/pull then in local FS
get available UAST types from SDK
for every driver, go though all /fixtures/*.sem.uast and count all uast:*

TODOs

for every driver, go though normalizer.go and count all usages of uast.*
get the list of all uast.* with go AST from SDK dynamically

Example report:

This change is

bzz commented 5 years ago

CI failure is un-related

- /site/using-babelfish/getting-started.html
  *  External link https://www.docker.com/community-edition failed: 403 No error

bzz commented 5 years ago

@dennwc thank you for the prompt review - all feedback has been addressed, it's ready for another round + would appreciate some feedback on a couple of open questions/improvement suggestions above.

bzz commented 5 years ago

Thank you for the reviews - all feedback addressed, @dennwc it's ready for another round.

dennwc commented 5 years ago

Github (again) does an awesome job of randomly hiding all comments including unresolved ones.

The change looks good, except for the driver filter (ForCurrentSDK and friends).

bzz commented 5 years ago

Somehow these must have slipped my mind, I'm sorry about that:

update formating https://github.com/bblfsh/documentation/pull/253#discussion_r286669264
filter out more drivers https://github.com/bblfsh/documentation/pull/253#discussion_r286915625

Will addd asap.

bzz commented 5 years ago

All feedback has been addressed, here is an example of the output:

Script uses go modules now, in order to be runnable without GOPATH and avoid confusion with SDK versions.

As counting of the code usage is not implemented yet, /0 is not reported by default any more - let me know what you think! Also it includes only 10 drivers that are actively maintained by us.