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

multiple file arguments on the command line, and the file= parameter in [bumpversion] #65

Open
rbeyer opened this issue Feb 20, 2019 · 3 comments

Comments

@rbeyer
Copy link

rbeyer commented Feb 20, 2019

I have file handling on my mind (#64) and I notice that there is a PendingDeprecationWarning on the ability to provide multiple files on the command line (and a similar warning on the undocumented files = parameter in the main [bumpversion] section of the config file--which accomplishes the same thing).

These PendingDeprecationWarnings are present in the upstream https://github.com/peritus/bumpversion. Why would that functionality be deprecated and is that still the plan?

@florisla
Copy link
Collaborator

I've read somewhere that the 'multiple files' functionality predates the .bumpversion.cfg config file support.

So the assumption seems to be that a proper configuration file makes things more flexible (especially given that you can set options differently per file) and covers all needs.

And I think it actually does cover all needs, save for the #64 functionality.

@c4urself
Copy link
Owner

c4urself commented Aug 24, 2019

I guess it makes sense to fully deprecate this once we address #64.

@florisla
Copy link
Collaborator

#111 also covers some of the use cases.

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

No branches or pull requests

3 participants