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

An empty working directory argument or option (deprecated) is ignored in minver-cli #589

Merged
merged 1 commit into from Aug 20, 2021

Conversation

adamralph
Copy link
Owner

Version(s)

1.0.0 - 2.5.0

To reproduce

Steps to reproduce the behaviour:

  1. Run minver-cli "" in a Windows command prompt (not PowerShell).

Expected behaviour

The command fails with

MinVer: error : Working directory '' does not exist.

Actual behaviour

The command succeeds.

Workarounds

Avoid passing an empty working directory argument.

Additional context

Fixing this is regarded as a breaking change, since current consumers may rely on the incorrect behaviour.

The behaviour is not reproducible in PowerShell because PowerShell appears to ignore "" and does not propagate it as an argument.

@adamralph adamralph added bug Something isn't working breaking This change could break current consumers labels Aug 20, 2021
@adamralph adamralph added this to the 3.0.0 milestone Aug 20, 2021
@adamralph adamralph merged commit e8deca7 into main Aug 20, 2021
@adamralph adamralph deleted the empty-working-directory branch August 20, 2021 21:57
@adamralph
Copy link
Owner Author

released in 3.0.0-alpha.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking This change could break current consumers bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant