Skip to content
This repository has been archived by the owner on Oct 11, 2023. It is now read-only.

Fix: Responses description not being included in the m4 output. #370

Merged
merged 5 commits into from
Dec 7, 2020
Merged

Fix: Responses description not being included in the m4 output. #370

merged 5 commits into from
Dec 7, 2020

Conversation

timotheeguerin
Copy link
Member

@timotheeguerin timotheeguerin commented Dec 7, 2020 •
edited
Loading

fix Azure/autorest#3704

  • Description included in success responses
  • Description included in errors responses

@azure-pipelines
Copy link

You may test this build by running autorest --reset and then either:

add --use: to the command line:

autorest --use:https://tinyurl.com/y522g8h6 ...

or use the following in your autorest configuration:

use-extension:
  "@autorest/modelerfour": "https://tinyurl.com/y522g8h6"

If this build is good for you, give this comment a thumbs up. (👍)

And you should run autorest --reset again once you're finished testing to remove it.

@Azure Azure deleted a comment from azure-pipelines bot Dec 7, 2020
@Azure Azure deleted a comment from azure-pipelines bot Dec 7, 2020
@Azure Azure deleted a comment from azure-pipelines bot Dec 7, 2020
@Azure Azure deleted a comment from azure-pipelines bot Dec 7, 2020
daviwil
daviwil approved these changes Dec 7, 2020
View reviewed changes
Copy link
Contributor

@daviwil daviwil left a comment

Choose a reason for hiding this comment

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

Looks good! Just one question

modelerfour/test/scenarios/expected/head/modeler.yaml Show resolved Hide resolved
@timotheeguerin timotheeguerin merged commit 2f4f58a into master Dec 7, 2020
@timotheeguerin timotheeguerin deleted the fix/missing-response-description branch December 7, 2020 22:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@daviwil daviwil daviwil approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@timotheeguerin @daviwil