-
Notifications
You must be signed in to change notification settings - Fork 56
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
fixed broken shell function #27
base: develop
Are you sure you want to change the base?
Conversation
That should only belong in the final build
I just made minor change, updating gulp components version. This helped me and runs tasks faster.
Updated components version
Thanks for sending this over :D I'll check it out today. It seems like a quick fix. I appreciate the help! |
Thanks again for updating this for everyone :) This is probably my fault, but I can't merge in the I could just replicate your changes myself, but I want to give you credit for the fix! Would you mind creating a new & fresh branch and putting your changes on there, then I can merge it in? Also, you don't need to comment out the updated code, you can just replace it instead of keeping the outdated shell work in there :) Thanks again, Ryan |
Hi amigo.
I'm afraid I don't know much about git, just mostly enough to manage my
little projects.
I'll try and figure out how to though and will learn something new this
weekend :) I've always wanted to contribute to a project :D
…On Fri, Sep 15, 2017 at 8:35 AM, Ryan Benson ***@***.***> wrote:
Thanks again for updating this for everyone :) This is probably my fault,
but I can't merge in the develop branch for this repo, because I was
using the develop branch to completely re-write this project. I'm not
sure if that was the best place to put it, but Harvest 2.0 is going to be a
pretty big shift from Harvest 1.0.
I could just replicate your changes myself, but I *want* to give you
credit for the fix! Would you mind creating a new & fresh branch and
putting your changes on there, then I can merge it in?
Also, you don't need to comment out the updated code, you can just replace
it instead of keeping the outdated shell work in there :)
Thanks again,
Ryan
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#27 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABXLRGFRHZwJYo7CIrAzcUmIDb0WJvzSks5siostgaJpZM4PVrLk>
.
|
//cleans our dist directory in case things got deleted
//gulp.task('clean', function() {
// return shell.task([
// 'rm -rf dist'
// ]);
// });
gulp.task('clean', shell.task([
'rm -rf dist'
]));
//create folders using shell
// gulp.task('scaffold', function() {
// return shell.task([
// 'mkdir dist',
// 'mkdir dist/fonts',
// 'mkdir dist/images',
// 'mkdir dist/scripts',
// 'mkdir dist/styles'
// ]
// );
// });
gulp.task('scaffold', shell.task([
'mkdir dist',
'mkdir dist/fonts',
'mkdir dist/images',
'mkdir dist/scripts',
'mkdir dist/styles'
]));