I don't have strong feelings about import ordering, but I do have strong feelings about extraneous lines in a diff, and some people have local setups that automatically sort imports in different ways. This PR introduces lint rules that should enforce deterministic ordering of imports and applies them across the code base so that in the future, we should have a common standard for import ordering and we shouldn't run into cases where a PR spontaneously introduces a new order.
Others may have strong feelings: they may want a different order, or not want these lint rules at all. Please speak up if so.
How to test
This shouldn't do anything but reorder a bunch of imports, so if CI passes, it's probably fine.
Review checklist
[x] as an author, I have thoroughly tested my changes and carefully followed the review guidelines
What it does
Fixes https://github.com/eclipse-cdt-cloud/vscode-memory-inspector/issues/98
I don't have strong feelings about import ordering, but I do have strong feelings about extraneous lines in a diff, and some people have local setups that automatically sort imports in different ways. This PR introduces lint rules that should enforce deterministic ordering of imports and applies them across the code base so that in the future, we should have a common standard for import ordering and we shouldn't run into cases where a PR spontaneously introduces a new order.
Others may have strong feelings: they may want a different order, or not want these lint rules at all. Please speak up if so.
How to test
Review checklist
Reminder for reviewers