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

Committing files generated by beforeBump task #141

Open
gxxcastillo opened this issue Nov 30, 2015 · 5 comments
Open

Committing files generated by beforeBump task #141

gxxcastillo opened this issue Nov 30, 2015 · 5 comments

Comments

@gxxcastillo
Copy link

beforeBump runs a build script in my library which generates 3 files. However, the 3 files are not being added to my commit. Is this expected behavior?

This is my config:

release: {
            options: {
                file: 'bower.json',
                additionalFiles: ['package.json'],
                beforeBump: ['build'],
                npm: false,
                npmtag: false
            }
}
@gxxcastillo
Copy link
Author

hmmm... looks like this was intentional?

    function add(){
      var files = config.files.join(' ');
      grunt.log.ok('files:' + files);
      return run('git add ' + files, ' staged ' + files);
    }

Any reason we wouldn't want to do a git add . --all?

Should there be an addAll config option?

@ritz078
Copy link

ritz078 commented Jan 4, 2016

👍 There no point to change your files before Release or bump if you are not commiting it.

@Klortho
Copy link

Klortho commented Mar 7, 2016

+1

@bago
Copy link

bago commented Mar 1, 2018

Or maybe alter the "add" command syntax to allow a list of files to be added... it would be even more intuitive. What's the point of the current "add" command? To add to the repository the package-lock or the changelog if they never existed before? How often does it happen?

@blendsdk
Copy link

I have a similar case where my build script needs to change some files. These files need to be committed also.

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

No branches or pull requests

5 participants