Skip to content

Linting: Remove the dependency comments lint rule. #73396

@youknowriad

Description

@youknowriad

One of the rules we have in our JS code base that we defined very early in the project lifecycle is the

  • External dependencies
  • WordPress dependencies
  • Internal dependencies.

Personally overtime, I noticed that they doesn't really add much value and only add friction. I'm proposing that we remove it entirely.

I know some projects also use alphabetical order for imports, while a bit better, I actually don't mind not having any rules here to ease refactoring, PRs and reviews.

Maybe the only rule that I'd have (and not sure how possible to do) is to write imports at the top of the file before actual code.

cc @WordPress/gutenberg-core

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions