Rename tools/analysis_plugin to tool/analysis_plugin

dart-lang / sdk

The Dart SDK, including the VM, JS and Wasm compilers, analysis, core libraries, and more.

https://dart.dev

BSD 3-Clause "New" or "Revised" License

10.2k stars 1.57k forks source link

Rename tools/analysis_plugin to tool/analysis_plugin #32865

Open MichaelRFairhurst opened 6 years ago

MichaelRFairhurst commented 6 years ago

The plural case triggers a pub warning on publish.

We can make them backwards compatible for probably a very long length of time if we have to, but otherwise probably any time after 2.0 stable would be a good time to remove support for 'tools/'.

cc @bwilkerson

original ticket https://github.com/dart-lang/angular_analyzer_plugin/issues/545

bwilkerson commented 6 years ago

@kevmoo Did you already have an issue for this, or was your request via e-mail?

kevmoo commented 6 years ago

Yup. Didn't file an issue, though.