-
-
Notifications
You must be signed in to change notification settings - Fork 1k
🐛 oragnizeImports doesn't organize bare export #7583
Copy link
Copy link
Labels
A-AnalyzerArea: analyzerArea: analyzerL-JavaScriptLanguage: JavaScript and super languagesLanguage: JavaScript and super languagesS-Bug-confirmedStatus: report has been confirmed as a valid bugStatus: report has been confirmed as a valid bugS-Needs discussionStatus: needs a discussion to understand criteriaStatus: needs a discussion to understand criteria
Metadata
Metadata
Assignees
Labels
A-AnalyzerArea: analyzerArea: analyzerL-JavaScriptLanguage: JavaScript and super languagesLanguage: JavaScript and super languagesS-Bug-confirmedStatus: report has been confirmed as a valid bugStatus: report has been confirmed as a valid bugS-Needs discussionStatus: needs a discussion to understand criteriaStatus: needs a discussion to understand criteria
Type
Fields
Give feedbackNo fields configured for Bug.
Environment information
What happened?
Running
oragnizeImportson the following code:provides the following results:
Note that the
exportwithout afromclause are not organized and merged.Expected result
A user could expect the following organized code:
This looks more complex that it seems. Because if we use the current rule logic, then we have to define an order between
export fromandexport. For example, if we decide thatexport fromis always placed first, then the resulting code could be:If it is a bug fix or a breaking change is up to debate.
We face some complains about the fact that
organizeImportsorganizes exports.Code of Conduct