-
Notifications
You must be signed in to change notification settings - Fork 12k
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
RFE: timestamp after compiler output #545
Comments
@hansl is this something that we should work on? |
I think it would involve having our own override to |
I think it can be done with some extra hack in https://github.com/angular/angular-cli/blob/master/lib/cli/index.js, but it also doesn't seem so important to me. |
Not important right now. Pushed after ng-conf. |
Not important unless you're trying to use it daily :/. Not being able to see if it's actually running and processing the current code is a real drag. |
It is running. Its just fast enough you can't see the changes. |
That's the gag - we've had it stop running more than once, and we like to have a visual indication that it hasn't done so again. |
#1455 changed the build step drastically, can you check if this is still needed? |
Sample of
|
It would be great to see a timestamp on ng build output .... is this being worked on by anyone? |
I've got a PR coming together, it should be in tomorrow. |
@gelliott181 What is the progress with your PR ? Is it bundled with #4057 ? |
This PR hit a snag after I dove down again and kept peeling away the calls.
To implement this either there needs to be a PR over in webpack-dev-server
as mentioned, or we need to write a custom webpack plugin. I think a plugin
is the best way to go, but I'd like to accumulate some more related
functionality before I add a whole new plugin to the build process and
maintenance list.
There isn't anywhere in the actual CLI code which outputs on serve builds,
it's all down the webpack hole. Someone could PR webpack-dev-server with a
couple new lines to do this, but that won't help here if it's rejected.
…On Sun, Jan 22, 2017 at 8:08 AM Carlo Dapor ***@***.***> wrote:
@gelliott181 <https://github.com/gelliott181> What is the progress with
your PR ? Is it bundled with #4057
<#4057> ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#545 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ADajDRkryOwnYqjtMKwwBneEsE4IkNw9ks5rU1SxgaJpZM4IQqjL>
.
|
There is a feature request for this in Webpack, if it gets in we can add it to the CLI: webpack/webpack#4718 |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
ng server "really should" output a timestamp at the end of its incremental compilation output, else it's impossible to confirm that it's doing anything unless one just happens to have it in view when it kicks in and can see it scroll by (it is normally hidden behind intelliJ).
e.g.
that would be a big usability improvement,
The text was updated successfully, but these errors were encountered: