Piotr Kuczynski

Results 253 comments of Piotr Kuczynski

Cool, thanks! I noticed that tests are not failing due to the issue I reported. You can also have a look at my demo repo directly to observe the issue......

Did you checked the example repo I provided? https://github.com/pkuczynski/eslint-plugin-import-1919-demo

This is actually true since quite a while, but nobody else reported an issue. Can you check again @ylluminate ?

This should not be required anymore if you run `rvm get master`

@TehShrike I need this too! So I basically want to merge only when the new object has a value (avoiding nulls overriding). Something like here: https://www.rubydoc.info/gems/deep_merge/1.2.1 ``` :merge_nil_values DEFAULT: false...

I have similar scenario with a false positive: ```js export const formatErrors = errors => errors.map((error) => { ... } ``` There is no promise involved in this piece of...

I agree with @jturel, that checking for a param name is a poor linting. However, I am flooded with work so I would not be able to come up with...

I would happily accept a PR if you have some time to work on it?