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

keep-outdated should be keep-optional #37

Open
rdohms opened this issue Oct 31, 2013 · 0 comments
Open

keep-outdated should be keep-optional #37

rdohms opened this issue Oct 31, 2013 · 0 comments

Comments

@rdohms
Copy link

rdohms commented Oct 31, 2013

I'm struggling a bit with the concept of keep-outdated. While it does perfectly what i need it to do, it does seem to describe the wrong concept for me.

In my .dist.yml file i actually ommit 2 parameters which are optional, since they have default values defined in the config file, however, if i do not keep them in the dist file, this script removes them.

So in this case, they are not outdated parameters as much as they are optional parameters, i think that would be better naming for it, or at least better motivation to keep those around, as outdated, the concept, means no longer used.

Just thought it worth the discussion.

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

1 participant