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.

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

Remove the OperationIdNounConflictingModelNames. rule #68

Open
dsgouda opened this issue Jul 17, 2017 · 2 comments
Open

Remove the OperationIdNounConflictingModelNames. rule #68

dsgouda opened this issue Jul 17, 2017 · 2 comments
Labels
p1 planned sdk-rule
Milestone
GA

Comments

@dsgouda
Copy link
Contributor

dsgouda commented Jul 17, 2017

Per conversation here, it looks like this rule is redundant and should be removed to avoid confusion for the users.
Documentation should be accordingly removed.
@dsgouda dsgouda added the p1 label Jul 17, 2017
@dsgouda dsgouda mentioned this issue Jul 17, 2017
Closed
@veronicagg
Copy link
Collaborator

@dsgouda if the arm flag makes us append the "Operations" string every time, I agree, this rule should not be reported. Have you confirmed this is the case? I'm not clear on where this behavior is code-wise, is it AutoRest or code generator specific?

@dsgouda
Copy link
Contributor Author

dsgouda commented Jul 19, 2017

Yes I have confirmed this and we should remove this soon to avoid false positives.

@sarangan12 sarangan12 added this to the GA milestone Feb 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
p1 planned sdk-rule
Projects
None yet
Milestone
GA
Development

No branches or pull requests
4 participants
@sarangan12 @dsgouda @vladbarosan @veronicagg