Skip to content
This repository has been archived by the owner on Apr 3, 2019. It is now read-only.

Clean-up gulp #27

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Clean-up gulp #27

wants to merge 1 commit into from

Conversation

leplatrem
Copy link
Contributor

No description provided.

@@ -25,12 +23,4 @@ gulp.task("dist", function() {
.pipe(gulp.dest(opt.outputFolder));
});

/**
Copy link
Member

Choose a reason for hiding this comment

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

Why would you like to do so? This is the way to go.

@Natim
Copy link
Member

Natim commented Nov 14, 2014

AFAIK I was adding the example here.

*/
gulp.task("deploy", ["dist"], function() {
gulp.src("./build/**")
.pipe(deploy("[email protected]:spiral-project/daybed.js.git"));
Copy link
Member

Choose a reason for hiding this comment

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

We're actually using gh-pages to deploy, I personally feel that's easier to have two branches, one for dev and one for something that's published.

This way, we chose to put what we want on the published pages and we can optimize it for the clients (minify the js for instance).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This gulp task was not very friendly with a non orphan branch.

Look at what I did for gh-pages in the source tree. That's why I do for many repos, and it works well.

Releasing consists in:

$ git checkout gh-pages
$ git merge vX.Y.Z
$ make dist
$ git commit --amend build/daybed.js
$ git push origin gh-pages

Copy link
Member

Choose a reason for hiding this comment

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

The aim of this task is to automate this and moreover be able to choose the tree for dev and for the gh-pages website.

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

Successfully merging this pull request may close these issues.

3 participants