`await-promise` error on mongoose project about lynt

[saadq]

Changed my mind, lol

#18

[danielpza]

With tslint I had something like

"no-floating-promises": [true, "DocumentQuery", "Query", "MongooseThenable"],

[saadq]

Unfortunately a lot of the TS errors were added in pretty hastily when I was creating the initial TSLint config, so there are a lot of overly oppressive rules right now that will be removed very soon. They are being kept track of in this issue.

I don't think it would be a good idea to try to come up with every case with for Promise-like stuff, so I think it might be just a good idea to remove this rule altogether and add it to that issue. Thoughts?

[danielpza]

Sounds good for me, maybe we could make a .lyntrc file like prettier to allow altering these rules a little

[saadq]

Fixed by #16. There is already support for a .lyntrc file as noted here, but it is only for the same stuff that the CLI offers like ignore and such. I don't really want to add support for configuration of rules, because I think that vastly takes away from the point of this project.

The one thing I may consider is adding a disable property to the settings which allows you to turn off any of the rules, but I think having something like enable wouldn't make sense – the rules that you would want to enable should already exist in lynt, and if they don't an issue should be raised about it or there is already a good reason why it shouldn't be in there.