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

--watch needs timestamp #21

Closed
bebraw opened this issue Dec 24, 2016 · 3 comments
Closed

--watch needs timestamp #21

bebraw opened this issue Dec 24, 2016 · 3 comments

Comments

@bebraw
Copy link

bebraw commented Dec 24, 2016

This isn't really a bug, but more a feature request. I'm not sure if this is the right forum for it?

Please can you add a timestamp to --watch so we can visually inspect if a build took place?

For example, sometimes --watch silently hangs, sometimes I save the wrong file etc. If the code editor and the terminal windows are not always visible at once, being able to glance at the timestamp of the most recent build is useful way to check if things are working.

This is akin to checking the timestamp of an executable to verify that the build succeeded and wrote the file you're expecting it to.

Thank you!

Moved from webpack/webpack#1499

@bebraw
Copy link
Author

bebraw commented Dec 24, 2016

This is literally

plugins: [
        function() {
            this.plugin('watch-run', function(watching, callback) {
                console.log('Begin compile at ' + new Date());
                callback();
            })
        }
    ]

as discussed in the comments.

This was referenced Dec 24, 2016
ericgundrum pushed a commit to ericgundrum/react-flip-card that referenced this issue Apr 29, 2017
With webpack 'hot-reload', there may be no feedback that webpack
noticed a file change and is serving the new file.
Searching for some way to configure webpack to report on each compile,
I found others discussing the issue in [webpack --watch needs timestamp]
(webpack/webpack#1499).

Their solution is a simple plugin. So simple it can be code in the
config, as reported in [webpack-cli --watch needs timestamp]
(webpack/webpack-cli#21) and included
in this commit.
@LDMFD
Copy link

LDMFD commented Sep 22, 2017

OP here. Thanks for implementing this!

@evenstensberg
Copy link
Member

💙💙

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants