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

Add Watch command #80

Merged
merged 6 commits into from
Apr 8, 2016
Merged

Conversation

LucioFranco
Copy link

Adds watch command and other things:

TODO:

  • Serve command now builds the site before serving it
  • Watch command builds the site then serves it then waits to see if a file has changed in the source dir and will rebuild on change Add a watch option #51
  • Add --port flag to pass a port to the server Add a --port option to cobalt serve #76
  • Simple usage message for build, watch, and serve

Let me know if there is anything else that should be added/changed.

@LucioFranco
Copy link
Author

@johannhof @tak1n would love if one of you could review this code. It's not fully done and I would love some suggestions for improvement. Thanks!

@johannhof
Copy link
Contributor

I'll take a look later today! :)

}
}
}
Err(e) => error!("[Notify Error]: {}", e),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we should std::process::exit(1) here

@johannhof
Copy link
Contributor

Luckily cobalt is quite quick right now so the just-rebuild-all approach will work for now, but we should definitely split up the different steps in cobalt.rs into public functions so that we can have at least a tiny bit of "incremental" compilation.

@johannhof
Copy link
Contributor

@LucioFranco this is awesome! It's definitely mergeable for me, I left a few minor nits for you anyway :)

@johannhof
Copy link
Contributor

👍

@johannhof johannhof merged commit 07f90ef into cobalt-org:master Apr 8, 2016
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

Successfully merging this pull request may close these issues.

2 participants