Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move analysis_server package to own github repo #22566

Closed
DartBot opened this issue Feb 25, 2015 · 4 comments
Closed

Move analysis_server package to own github repo #22566

DartBot opened this issue Feb 25, 2015 · 4 comments
Labels
analyzer-server area-analyzer Use area-analyzer for Dart analyzer issues, including the analysis server and code completion. P2 A bug or feature request we're likely to work on type-enhancement A request for a change that isn't a bug

Comments

@DartBot
Copy link

DartBot commented Feb 25, 2015

This issue was originally filed by @seaneagan


I understand that the dart repo will be moving to github soon anyways, but it will still make sense for the analysis server to have it's own repo there:

https://code.google.com/p/dart/source/browse/#svn%2Fbranches%2Fbleeding_edge%2Fdart%2Fpkg%2Fanalysis_server

This would make it much easier for me to add pubspec-related functionality from den to it, see:

seaneagan/den#17

@bwilkerson
Copy link
Member

I expect that we will do this fairly soon.


Removed Priority-Unassigned label.
Added Priority-Medium, Area-Analyzer, Analyzer-Server, Triaged labels.

@DartBot
Copy link
Author

DartBot commented Apr 20, 2015

This comment was originally written by @seaneagan


Any milestone estimate for this one?

@bwilkerson
Copy link
Member

No, and it's less clear to me that we will be doing this.

@DartBot DartBot added Type-Enhancement area-analyzer Use area-analyzer for Dart analyzer issues, including the analysis server and code completion. analyzer-server labels Apr 20, 2015
@kevmoo kevmoo added P2 A bug or feature request we're likely to work on type-enhancement A request for a change that isn't a bug and removed triaged labels Mar 1, 2016
@bwilkerson
Copy link
Member

I think it's safe to say that we won't be doing this. Having code outside the sdk repo that is tightly coupled to code inside it reduces our velocity.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
analyzer-server area-analyzer Use area-analyzer for Dart analyzer issues, including the analysis server and code completion. P2 A bug or feature request we're likely to work on type-enhancement A request for a change that isn't a bug
Projects
None yet
Development

No branches or pull requests

3 participants