Bryan Mishkin

icon indicating a website link https://www.linkedin.com/in/bmish/

icon company @vercel icon location New York City and San Francisco icon location Software Engineer @vercel. Passionate about the web, open source, and dev tools. Former @square, @eslint team, @emberjs linting.

Results 359 comments of Bryan Mishkin

Change Request: Enable `reportUnusedDisableDirectives` config option by default

This will require an RFC. I can work on putting together an RFC based on what I proposed in my previous comment.

Change Request: Enable `reportUnusedDisableDirectives` config option by default

I have opened an RFC for this: * https://github.com/eslint/rfcs/pull/100

Change Request: Enable `reportUnusedDisableDirectives` config option by default

Part 1 is done but I'm going to reopen this until part 2 of the RFC is done: > Part 1: add support for the new severity levels * https://github.com/eslint/eslint/pull/17212...

Change Request: Enable `reportUnusedDisableDirectives` config option by default

Yes I started part 2 in: * #17879.

Rules that need autofixers

@tgvrssanthosh I created a list of all the Ember array functions/properties we need to handle to track progress. Thanks for working on this.

Rules that need autofixers

@Turbo87 it's a fair concern. We made the call to proceed with the autofixer because: 1. The autofixer is obviously dramatically more useful in terms of mass migration compared to...

Even more false positives with `no-array-prototype-extensions` rule

Just to spell out one potential heuristic we could add for this related example: ```js async _fetchCategories(query) { const response = await this.store.query('connect/category', query); return response.toArray(); // false positive }...

`no-deeply-nested-dependent-keys-with-each` rule should apply to computed macros that return an @each

Will need a volunteer to address it.

chore(CI): simplify testing strategy

Can you summarize what's changed?

chore(CI): simplify testing strategy

This should fix one of the issues: #315