Skip to content

Conversation

@simo9000
Copy link
Contributor

@simo9000 simo9000 commented Jan 3, 2017

PR checklist

  • Read the contribution guildelines.
  • Ran the shell/batch script under ./bin/ to update Petstore sample so that CIs can verify the change. (For instance, only need to run ./bin/{LANG}-petstore.sh and ./bin/security/{LANG}-petstore.sh if updating the {LANG} (e.g. php, ruby, python, etc) code generator or {LANG} client's mustache templates)
  • Filed the PR against the correct branch: master for non-breaking changes and 2.3.0 branch for breaking (non-backward compatible) changes.

Description of the PR

Updating the Nancy reference to version 1.4.3 in the project and packages config mustache files

@simo9000 simo9000 mentioned this pull request Jan 3, 2017
<ItemGroup>
<Reference Include="Nancy, Version=1.4.1.0, Culture=neutral, processorArchitecture=MSIL">
<HintPath>..\..\packages\Nancy.1.4.1\lib\net40\Nancy.dll</HintPath>
<Reference Include="Nancy, Version=1.4.2.0, Culture=neutral, processorArchitecture=MSIL">
Copy link
Contributor

Choose a reason for hiding this comment

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

Is 1.4.2 on this line correct? Or is this a typo?

Copy link
Contributor Author

@simo9000 simo9000 Jan 3, 2017

Choose a reason for hiding this comment

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

I know it's weird, but I did a nuget update-package on the petstore example and this is exactly how it modified the project file. It compiled so i didn't ask any questions.

Copy link
Contributor

Choose a reason for hiding this comment

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

That is weird. Nuget does that sometimes, I just wanted to check that it wasn't a typo. I think this version is the version from nuspec while the following line is from assembly info.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah and it's a major pain to keep the two consistent with very little benefit.

@wing328 wing328 merged commit aa1dc0f into swagger-api:master Jan 5, 2017
@wing328 wing328 added this to the v2.2.2 milestone Jan 5, 2017
@wing328
Copy link
Contributor

wing328 commented Jan 5, 2017

@simo9000 thanks for the PR, which has been merged into master.

@simo9000 simo9000 deleted the nancyUpdate branch January 5, 2017 17:11
@wing328 wing328 changed the title Updated Nancy from 1.4.1 to 1.4.3 [C#][Nancy] Updated Nancy from 1.4.1 to 1.4.3 Feb 20, 2017
@simo9000 simo9000 restored the nancyUpdate branch March 17, 2017 21:40
davidgri pushed a commit to davidgri/swagger-codegen that referenced this pull request May 11, 2017
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.

3 participants