Remove .editorconfig

[flevi29]

Pull Request

Why?

.editorconfig was added on the very first commit of this repository, but since then prettier has replaced it. Except it wasn't removed. There's no point in having multiple configurations for formatting. Related #1659, in which I wanted to remove all custom options, but didn't know why things still didn't work as they should, and now I know that prettier also takes .editorconfig into consideration.

PR checklist

Please check if your PR fulfills the following requirements:

• Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
• Have you read the contributing guidelines?
• Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!

[flevi29]

I am not applying the fmt:fix, because it will be a lot of changes, for now. Once this is approved I can apply it, or better yet a member should apply it, so that they can trust that no malicious code is hidden in the changes and they don't even have to review the changes that way.

[codecov]

Codecov Report

Attention: Patch coverage is 95.87629% with 4 lines in your changes missing coverage. Please review.

Project coverage is 97.11%. Comparing base (8a659a5) to head (96ad083).
Report is 22 commits behind head on main.

Files with missing lines	Patch %	Lines
src/indexes.ts	85.71%	2 Missing ⚠️
src/http-requests.ts	93.33%	1 Missing ⚠️
tests/utils/meilisearch-test-utils.ts	93.75%	1 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1696   +/-   ##
=======================================
  Coverage   97.11%   97.11%           
=======================================
  Files          21       21           
  Lines         831      831           
  Branches       84       84           
=======================================
  Hits          807      807           
- Misses         23       24    +1     
+ Partials        1        0    -1     

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

[curquiza]

I approve, but indeed the tests should be fixed before merging 😇

[flevi29]

I approve, but indeed the tests should be fixed before merging 😇

Should I do it? It will be a lot of changes, or so I presume.
EDIT: I ran it

[curquiza]

Thank you @flevi29
Same, we don't merge this PR until the coming release on Monday. I don't want to block: #1682

[curquiza]

bors merge

[meili-bors]

Build succeeded:

• integration-tests (Node.js 18)
• integration-tests (Node.js 20)
• style-check
• types-check