dstopic icon indicating copy to clipboard operation
dstopic copied to clipboard
verified publisher icon entropic-dev

Scanner

Open MilanLoveless opened this issue 6 years ago • 2 comments

Here is our first pass at the require/import scanner for this issue: https://discourse.entropic.dev/t/looking-for-help-rust-based-js-import-scanner/218

TODO:

  • rename functions so they actually make sense. Currently we split off new functions every time we went another level deeper in match statements.
  • document all the things!
  • more tests!
  • figure out whether to expose ressa::Error as part of our Result<> return type or if we should remap it to our own internal error type.

MilanLoveless avatar Jul 24 '19 22:07 MilanLoveless

This pull request has been mentioned on Entropic. There might be relevant details there:

https://discourse.entropic.dev/t/looking-for-help-rust-based-js-import-scanner/218/47

static-raccoon avatar Jul 25 '19 00:07 static-raccoon

Hey @MilanLoveless! I know things have been quiet for a while, but it looks like we're ramping up again. As part of that, I'm organizing our repo stuff a bit, and one of those changes is I'm going to merge the dstopic repo into the ds repo so we only need to clone one monorepo for both "sides" of ds. Do you mind recreating this PR against that repo? I've already move the code over there.

zkat avatar Nov 09 '19 05:11 zkat