Create a high level library of helper functions about angular-hint HOT 9 CLOSED

–> https://github.com/angular/angular-hint-log

here ya go

from angular-hint.

Awesome! Thank you @btford. I started working on the messaging repo #17

I'm wondering if we should also have a shared place for functions like @caguillen214's spell checking function? I think several of his repos make use of that functionality? Or maybe that library (ddLib I think?) should be a separate node module akin to dom-interceptor.js so that it could be loaded separately?

from angular-hint.

I'm wondering if we should also have a shared place for functions like @caguillen214's spell checking function?

Yeah probably.

We should also check that something like @caguillen214's function doesn't already exist in npm.

from angular-hint.

I looked around the npm website and found https://www.npmjs.org/package/natural, which has a various number of string comparison functions. However, it has a lot more than we need (just the levenshtein distance) so should we really add the whole npm module if we're not going to use the whole thing?

from angular-hint.

We definitely want something more minimal, as opposed to a "kitchen-sink" module.

npm search levenshtein seems to have many options. check out some of those.

from angular-hint.

I've looked at some of the levenshtein packages and even the ones that are just levenshtein packages provide a few other methods that we really dont need. These packages are significantly more light weight than the 'kitchen-sink' packages.

We only need to calculate the LD and right now we have a function that does that, is it the best option to include a package from npm?

from angular-hint.

https://github.com/angular/angular-hint-events/issues/2 @caguillen214

from angular-hint.

#25 duplicate discussion

from angular-hint.

@caguillen214 refactored the duplicate functionality to use the suggest-it npm module.

from angular-hint.