Skip to content

Provide support for reading a single parameter value from file - #738

Merged
Johan Stenberg (ex-msft) (johanste) merged 33 commits into
Azure:masterfrom
johanste:atfile
Aug 31, 2016
Merged

Johan Stenberg (ex-msft) (johanste) merged 33 commits into
Azure:masterfrom
johanste:atfile

Conversation

@johanste

Copy link
Copy Markdown
Contributor

Values prefixed with @ will be replaced with the content of the file immediately following the @-sign.

The pseudo-name '-' is used to indicate stdin.

Values prefixed with @ will be replaced with the content of the file immediately following the @-sign.

The pseudo-name '-' is used to indicate stdin.
@tjprescott

Copy link
Copy Markdown
Member

:shipit:

def _load_file(path):
try:
if path == '-':
content = sys.stdin.read()

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we throw an error if more than one - is specified?

* Convert all updates to cli_generic_update_commands.

* Code review fixes.
* Eliminates anti-pattern of creating CliArgumentTypes inline during parameter registration.

* Minor fixes.
* Fix issue Azure#680.

* Initial version of help dump. Logic to extract summary.

* Fix issues Azure#365 and Azure#447.

* Code review fixes.

* Code review fixes.
* 'az configure' experience

- Allows user to set default output type.
- Infrastructure for CLI configurations also implemented.

* Add license header & fix deprecated method

* Wrap SafeConfigParser instead of extending from it

* Code review feedback changes
* Table format revisions

- Table output is generic and automatically extracts fields from the result.
- No longer use simple_output_query as it was specific for commands and as packages will support multiple API versions, this solution is no longer feasible.

* Support callable again for table format after discussion

When a callable is set for a command, it will be used as long as there is no query active.
If there's a query active, the callable will not be used so the user has to specify a full query that can generate an appropriate table.
* Remove 'starting' message for long running operations

* Change name of poll_interval_ms as ambiguous and also change logger messages that referred to this interval
@derekbekoe

Copy link
Copy Markdown
Member

:shipit:

Kevin Zhao (xscript) and others added 10 commits August 29, 2016 18:21
* Add commands for Azure IoT

* 1.Include files into azure-cli.pyproj; 2. Update license info in setup.py

* add missing pylint directive

* resolve review comments

* resolve comments

* fix typo
* Fix issues Azure#728 and Azure#686

* Code review fixes.
Values prefixed with @ will be replaced with the content of the file immediately following the @-sign.

The pseudo-name '-' is used to indicate stdin.
@johanste
Johan Stenberg (ex-msft) (johanste) merged commit ceff1c4 into Azure:master Aug 31, 2016
Kerwin(Kaihui) Sun (00Kai0) pushed a commit to 00Kai0/azure-cli that referenced this pull request Apr 7, 2021
* [db-up] Set the max CLI core version as 2.0.66

* [db-up] Publish 0.1.12

* [db-up] Remove config_parser
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants