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

RFE: timestamp after compiler output #545

Closed
sgbeal opened this issue Apr 27, 2016 · 15 comments · Fixed by #6989
Closed

RFE: timestamp after compiler output #545

sgbeal opened this issue Apr 27, 2016 · 15 comments · Fixed by #6989
Assignees
Labels
effort2: medium (days) needs: more info Reporter must clarify the issue P5 The team acknowledges the request but does not plan to address it, it remains open for discussion

Comments

@sgbeal
Copy link

sgbeal commented Apr 27, 2016

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.

file added app\components\msisdn-search\msisdn-search-form.component.ts___jb_bak___
...    
Build successful - 532ms.

Slowest Trees                                 | Total
----------------------------------------------+---------------------
BroccoliTypeScriptCompiler                    | 275ms
...

2016-04-27 09:17 <<<<------ here

that would be a big usability improvement,

@jkuri
Copy link
Contributor

jkuri commented Apr 28, 2016

@hansl is this something that we should work on?

@filipesilva
Copy link
Contributor

I think it would involve having our own override to Builder, so it's not that trivial.

@jkuri
Copy link
Contributor

jkuri commented Apr 28, 2016

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.

@hansl
Copy link
Contributor

hansl commented Apr 28, 2016

Not important right now. Pushed after ng-conf.

@sgbeal
Copy link
Author

sgbeal commented Apr 28, 2016

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.

@jkuri
Copy link
Contributor

jkuri commented Apr 28, 2016

It is running. Its just fast enough you can't see the changes.

@sgbeal
Copy link
Author

sgbeal commented Apr 28, 2016

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.

@filipesilva filipesilva added type: enhancement effort2: medium (days) P5 The team acknowledges the request but does not plan to address it, it remains open for discussion labels Jun 4, 2016
@filipesilva
Copy link
Contributor

#1455 changed the build step drastically, can you check if this is still needed?

@filipesilva filipesilva added the needs: more info Reporter must clarify the issue label Jul 27, 2016
@filipesilva
Copy link
Contributor

Sample of ng serve output:

kamik@T460p MINGW64 /D/sandbox/webpack-project (master)
$ ng serve
(node:12676) fs: re-evaluating native module sources is not supported. If you are using the graceful-fs module, please update it to a more recent version.
 10% building modules 1/1 modules 0 active
*
*
 NG Live Development Server is running on http://localhost:4200.
*
5368ms building modules
12ms sealing
0ms optimizing
0ms basic module optimization
18ms module optimization
2ms advanced module optimization
7ms basic chunk optimization
2ms chunk optimization
0ms advanced chunk optimization
0ms module and chunk tree optimization
45ms module reviving
8ms module order optimization
4ms module id optimization
7ms chunk reviving
1ms chunk order optimization
11ms chunk id optimization
29ms hashing
2ms module assets processing
52ms chunk assets processing
4ms additional chunk assets processing
1ms recording
0ms additional asset processing
104ms chunk asset optimization
691ms asset optimization
12ms emitting
Hash: da0621dd0d54cc99ff06
Version: webpack 2.1.0-beta.18
Time: 6387ms
              Asset       Size  Chunks             Chunk Names
     main.bundle.js    2.18 MB    0, 2  [emitted]  main
polyfills.bundle.js     225 kB    1, 2  [emitted]  polyfills
          inline.js    4.96 kB       2  [emitted]  inline
           main.map    2.58 MB    0, 2  [emitted]  main
      polyfills.map     287 kB    1, 2  [emitted]  polyfills
         inline.map    5.13 kB       2  [emitted]  inline
         index.html  502 bytes          [emitted]
         .gitignore    0 bytes          [emitted]
         .npmignore    0 bytes          [emitted]
Child html-webpack-plugin for "index.html":
         Asset     Size  Chunks       Chunk Names
    index.html  2.44 kB       0
webpack: bundle is now VALID.

@mkelley82
Copy link

It would be great to see a timestamp on ng build output .... is this being worked on by anyone?

@celliott181
Copy link
Contributor

I've got a PR coming together, it should be in tomorrow.

@catull
Copy link
Contributor

catull commented Jan 22, 2017

@gelliott181 What is the progress with your PR ? Is it bundled with #4057 ?

@celliott181
Copy link
Contributor

celliott181 commented Jan 22, 2017 via email

@filipesilva
Copy link
Contributor

There is a feature request for this in Webpack, if it gets in we can add it to the CLI: webpack/webpack#4718

@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
effort2: medium (days) needs: more info Reporter must clarify the issue P5 The team acknowledges the request but does not plan to address it, it remains open for discussion
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants