Azure / Azure/autorest.powershell

Poor code readability and need simplify class name

Open
#523 0 comments 0 reactions 0 assignees View on GitHub
discussion
Dominant language
C#
Stars
123
Forks
99
PR merge metrics
No merged PRs in 30d

Description

Hi team,
The autorest@beta is so good. But I think that the powershell code generated by autorest --powershell has poor readability.

For example: this is a line generated by autorest@beta
```
await ((Microsoft.Azure.Commands.Management.ProjectArcadia.Runtime.IEventListener)this).Signal(Microsoft.Azure.Commands.Management.ProjectArcadia.Runtime.Events.CmdletProcessRecordAsyncStart); if( ((Microsoft.Azure.Commands.Management.ProjectArcadia.Runtime.IEventListener)this).Token.IsCancellationRequested ) { return; }
```
I think it is two statements and can split into two lines.

Besides, my visual studio shows that `Microsoft.Azure.Commands.Management.ProjectArcadia.Runtime.IEventListener` can be simplified to `Runtime.IEventListener`.

Hope you can solve some problem about code readability.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.