Skip to content

Automatically reload transformers when the pubspec changes #19163

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

Closed
stevemessick opened this issue Jun 3, 2014 · 12 comments
Closed

Automatically reload transformers when the pubspec changes #19163

stevemessick opened this issue Jun 3, 2014 · 12 comments
Assignees
Labels
closed-not-planned Closed as we don't intend to take action on the reported issue P1 A high priority bug; for example, a single project is unusable or has many test failures type-enhancement A request for a change that isn't a bug
Milestone

Comments

@stevemessick
Copy link
Contributor

[user feedback]

Adding transformers to a pubspec requires me to manually kill pub serve.

I added the polymer transformer, but was still getting errors in my app resulting from not having the transformer. Killing pub serve fixed the problem.
////////////////////////////////////////////////////////////////////////////////////
Editor: 1.5.0.dev_02_00 (2014-05-26)
OS: Mac OS X - x86_64 (10.9.2)
JVM: 1.6.0_65

projects: 26

open dart files: 8

auto-run pub: true
localhost resolves to: 127.0.0.1
mem max/total/free: 1983 / 1983 / 237 MB
thread count: 55
index: 5630817 relationships in 418107 keys in 4146 sources

SDK installed: true
Dartium installed: true

@nex3
Copy link
Member

nex3 commented Jun 3, 2014

Please include the --verbose log of the errors you're receiving.


Added NeedsInfo label.

@justinfagnani
Copy link
Contributor

This was reported by me via the Editor feedback. I just happened to see it roll by the issues list, but you should know there's no way for Steve to get you more info from [user feedback].

Anyway, I didn't receive any errors from pub, I received errors from Polymer because pub didn't run the transformer I added, it just happily used the stale configuration from the previous version of my pubspec.yaml.


Added Triaged label.

@nex3
Copy link
Member

nex3 commented Jun 4, 2014

Removed Priority-Unassigned label.
Added Priority-Low label.
Changed the title to: "Automatically reload transformers when the pubspec changes".

@nex3
Copy link
Member

nex3 commented Jun 4, 2014

Issue #19214 has been merged into this issue.

@nex3
Copy link
Member

nex3 commented Jun 26, 2014

Marked this as blocking #14206.

@DartBot
Copy link

DartBot commented Aug 23, 2014

This comment was originally written by @zoechi


Currently pub serve kills itself when the pubspec.yaml of the project it currently serves was changes (I assume it is pub serve or is it DartEditor who kills pub serve?)

I had the case today where I had to add a transformer to a dependency (path dependency). In this case pub serve wasn't killed and it took me a while to figure out why the configuration change didn't take effect.

See also https://code.google.com/p/dart/issues/detail?id=20673

@munificent
Copy link
Member

I assume it is pub serve or is it DartEditor who kills pub serve?

It's an Editor feature. Pub serve itself doesn't currently check for this (though we would like it to).

I had the case today where I had to add a transformer to a dependency (path dependency). In this case pub serve wasn't killed and it took me a while to figure out why the configuration change didn't take effect.

Ah, yes. I don't think the Editor considers that case.


Removed Area-Pub label.
Added Area-Editor label.

@sethladd
Copy link
Contributor

Seems reasonable to restart pub serve when a change to pubspec.yaml is detected.


Set owner to @keertip.
Removed Priority-Low label.
Added Priority-Hg label.

@sethladd
Copy link
Contributor

Added this to the 1.7 milestone.
Removed Priority-Hg label.
Added Priority-High label.

@munificent
Copy link
Member

Seems reasonable to restart pub serve when a change to pubspec.yaml is detected.

My understanding is that the Editor does that now when it's your own pubspec that changed. What it doesn't do, that gzoechi is talking about, is monitor the pubspecs of your path dependencies for changes.

@sethladd
Copy link
Contributor

ah, thanks for the clarification.

@sethladd
Copy link
Contributor

sethladd commented Sep 4, 2014

Would it be reasonable for pub serve to reload itself when pubspec.yaml changes? This way, it's not an Editor responsibility and WebStorm, Sublime, and other plugins will get this for free.

Going to close this for Editor, but would love to see a discussion around reloading and pub serve.


Added NotPlanned label.

@stevemessick stevemessick added Type-Enhancement P1 A high priority bug; for example, a single project is unusable or has many test failures closed-not-planned Closed as we don't intend to take action on the reported issue labels Sep 4, 2014
@stevemessick stevemessick added this to the 1.7 milestone Sep 4, 2014
@kevmoo kevmoo added type-enhancement A request for a change that isn't a bug and removed type-enhancement labels Mar 1, 2016
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed-not-planned Closed as we don't intend to take action on the reported issue P1 A high priority bug; for example, a single project is unusable or has many test failures type-enhancement A request for a change that isn't a bug
Projects
None yet
Development

No branches or pull requests

8 participants