Skip to content

Fixed resolving relative swaggers#1822

Merged
olydis merged 10 commits intoAzure:masterfrom
amarzavery:fixresolve
Feb 17, 2017
Merged

Fixed resolving relative swaggers#1822
olydis merged 10 commits intoAzure:masterfrom
amarzavery:fixresolve

Conversation

@amarzavery
Copy link
Copy Markdown
Contributor

No description provided.

@amarzavery
Copy link
Copy Markdown
Contributor Author

Fixes #1778. Have tested this locally

@lmazuel
Copy link
Copy Markdown
Member

lmazuel commented Feb 14, 2017

@amarzavery
Copy link
Copy Markdown
Contributor Author

Yup. I tested my fix on that spec and Autorest does not give an error. Please feel free to pull from my fork and give it a try. This should definitely unblock you.

@lmazuel
Copy link
Copy Markdown
Member

lmazuel commented Feb 14, 2017

@amarzavery I confirm it's ok from your fork!!! Thanks :)

@amarzavery
Copy link
Copy Markdown
Contributor Author

@lmazuel - Thank you. Sorry for messing it up the first time.

@olydis
Copy link
Copy Markdown
Contributor

olydis commented Feb 15, 2017

@amarzavery Note the changed folder structure (now uniform across all projects). Please move your added swagger files to the right spot (Resource/...) and CI should be happy :-)

@amarzavery
Copy link
Copy Markdown
Contributor Author

Sure sir. will do that.

@olydis
Copy link
Copy Markdown
Contributor

olydis commented Feb 15, 2017

Thank you sir.

@olydis olydis merged commit 96218a0 into Azure:master Feb 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants