[chore]: minimize eslint setting overrides

[anikdhabal]

Which problem is this PR solving?

Resolves #1258

Short description of the changes

made the root eslintrc also extend the recommended config from @typescript-eslint.

[yurishkuro]

What could be the testing methodology for this PR? I.e., just having a green CI seems insufficient as it doesn't prove that a group of files are not getting excluded from linting after the change.

[anikdhabal]

What could be the testing methodology for this PR? I.e., just having a green CI seems insufficient as it doesn't prove that a group of files are not getting excluded from linting after the change.

I've tested it locally using the "eslint", but I don't think it is relevant. Any suggestion?

[yurishkuro]

maybe introducing some errors in the files and observing that linting errors are reported

[anikdhabal]

@yurishkuro can you review it once?

[yurishkuro]

thanks!

[yurishkuro]

looks like linting is failing https://github.com/jaegertracing/jaeger-ui/actions/runs/5836530756/job/15830302191?pr=1565

[anikdhabal]

looks like linting is failing https://github.com/jaegertracing/jaeger-ui/actions/runs/5836530756/job/15830302191?pr=1565

Yes, trying to fix it.

[codecov]

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.05% 🎉

Comparison is base (002932d) 95.96% compared to head (390ccfb) 96.01%.
Report is 73 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1565      +/-   ##
==========================================
+ Coverage   95.96%   96.01%   +0.05%     
==========================================
  Files         242      241       -1     
  Lines        7557     7562       +5     
  Branches     1984     1987       +3     
==========================================
+ Hits         7252     7261       +9     
+ Misses        305      301       -4     

Files Changed	Coverage Δ
...mponents/TraceDiff/TraceDiffHeader/CohortTable.tsx	100.00% <ø> (ø)
...s/TracePage/TraceTimelineViewer/ListView/index.tsx	95.45% <ø> (ø)
packages/jaeger-ui/src/reducers/metrics.mock.js	100.00% <ø> (ø)
...ndencies/Graph/DdgNodeContent/calc-positioning.tsx	100.00% <100.00%> (ø)
...aeger-ui/src/components/DeepDependencies/index.tsx	100.00% <100.00%> (ø)
...r-ui/src/components/Monitor/ServicesView/index.tsx	97.34% <100.00%> (ø)
.../components/TracePage/TraceTimelineViewer/duck.tsx	98.68% <100.00%> (ø)
...jaeger-ui/src/components/TracePage/scroll-page.tsx	100.00% <100.00%> (ø)
.../src/components/common/DetailsCard/DetailTable.tsx	100.00% <100.00%> (ø)
...ages/jaeger-ui/src/components/common/TraceName.tsx	100.00% <100.00%> (ø)
... and 7 more

... and 9 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[anikdhabal]

@yurishkuro fixed

[anshgoyalevil]

286 eslint warnings were introduced with PR
https://github.com/jaegertracing/jaeger-ui/actions/runs/5907286634/job/16024953128#step:6:636
Was this the expected behavior?

[yurishkuro]

286 eslint warnings were introduced with PR ... Was this the expected behavior?

Not ideal. Perhaps we can fine tune the settings, on case by case basis.