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.

Sign up for GitHub

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

Jump to bottom

Added a small utility method to display a warnings. #16441

Merged

maxcoulombe merged 5 commits into main from vault-2165-args-warning

Jul 27, 2022

Commits on Jul 25, 2022

Added a small utility method to display a warnings.
```
Will print warning if flag is passed after arguments e.g.
vault <command> -a b -c
In this example -c will be interpreted as an argument which may be misleading
```
maxcoulombe committed Jul 25, 2022
Copy the full SHA

1f2aa60 View commit details

Browse the repository at this point in the history
renamed changelog file

maxcoulombe committed Jul 25, 2022

Copy the full SHA

e2c6203 View commit details

Browse the repository at this point in the history

Commits on Jul 26, 2022

* using reference to UI instead of print to display the warning

maxcoulombe committed Jul 26, 2022

Copy the full SHA

2c65cc1 View commit details

Browse the repository at this point in the history
Merge remote-tracking branch 'origin/main' into vault-2165-args-warning

maxcoulombe committed Jul 26, 2022

Copy the full SHA

1399f81 View commit details

Browse the repository at this point in the history

Commits on Jul 27, 2022

* modified changelog to be more user-focused
```
* returning string value instead of using output buffer
* rephrased warning message for clarity
```
maxcoulombe committed Jul 27, 2022
Copy the full SHA

4696aae View commit details

Browse the repository at this point in the history