Build: Tag demos.jquerymobile.com repo on release #7193

Merged
merged 1 commit into from Feb 28, 2014

Projects

None yet

2 participants

Member
gseguin commented Feb 28, 2014

Fixes gh-7191

@arschmitz arschmitz commented on an outdated diff Feb 28, 2014
build/release.js
@@ -55,10 +55,11 @@ module.exports = function( Release ) {
process.chdir( repo );
Release.git( "add ." , "Error adding files." );
Release.git( "commit -m '" + commitMessage + "'" , "Error commiting demos files." );
- if ( !Release.isTest ) {
- console.log( "Pushing to github..." );
- Release.git( "push", "Error pushing demos to github." );
- }
+ Release.exec( "npm version patch" );
+ if ( !Release.isTest ) {
+ console.log( "Pushing to github..." );
+ Release.git( "push --tags", "Error pushing demos to github." );
arschmitz
arschmitz Feb 28, 2014 Owner

i think this should be push --tags origin master like we were talking about if you can verify this so everything stays in sync that would be awesome

Owner

if you can just address that one line one way or another 👍

Member
gseguin commented Feb 28, 2014

I had it in my workspace that way, forgot to commit. There it is.

Member
gseguin commented Feb 28, 2014

And now even with the right whitespace

@gseguin gseguin merged commit b9b8fcf into master Feb 28, 2014

1 check passed

default Code Climate analyzed this pull request.
Details
@gseguin gseguin deleted the issue-7191 branch Feb 28, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment