-
Notifications
You must be signed in to change notification settings - Fork 26
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
Support task runner #129
Comments
You mean, making Grunt/gulp/broccoli/etc plugins to wrap Harmonic? Once we have an working API, wrapping Harmonic with build tool/task runner plugins will be easy or even unnecessary. |
Let me try to explain what i meant. Assuming a website with using Grunt as task runner, which have concatenation, lint and minify tasks. As i said, i don't know yet the best way to deal with. |
If the developer wants to integrate Harmonic into his gulp/Grunt/etc. workflow, I believe the Harmonic API is best suited for the task. E.g.: gulp.task('build', ['lint'], function(cb) {
new Harmonic('path/to/project').build().then(function(){ cb(); }, cb);
});
gulp.task('lint', function() {
// ...
}); Then executing Is this what you meant? |
No, hahaa. For websites, i don't think this is a good approach. Take a look: harmonic init // start
harmonic new_post "my post" // some post
harmonic new_page "some page" // some page Here, we have an article and a page. What i meant in the beginning is that Harmonic could support a way for developers tell what tasks to run, or what coomand to run, so he can still just running: harmonic run And Harmonic will execute Grunt/Gulp/Whatever tasks. |
Ohh, I guess I understand it now. So the use cases are:
Did I get it right this time? For use case 1, perhaps the currently selected theme could hook into the plugins API. For use case 2, the developer could |
All modern website are built using some kind of
task-runner
.Harmonic must to support
task-runner
integration in some way.Maybe a custom CLI command that Harmonic can run after or before the build.
I don't know yet what is the best approach here.
The text was updated successfully, but these errors were encountered: