Skip to content

[nancyfx] Issue #5196: make package GUID nonrandom for nancyfx samples#5209

Merged
wing328 merged 2 commits intoswagger-api:masterfrom
ePaul:5196-make-package-uuid-nonrandom-for-nancyfx-samples
Mar 27, 2017
Merged

[nancyfx] Issue #5196: make package GUID nonrandom for nancyfx samples#5209
wing328 merged 2 commits intoswagger-api:masterfrom
ePaul:5196-make-package-uuid-nonrandom-for-nancyfx-samples

Conversation

@ePaul
Copy link
Copy Markdown
Contributor

@ePaul ePaul commented Mar 26, 2017

PR checklist

  • Read the contribution guidelines.
  • Ran the shell/batch script under ./bin/ to update Petstore sample so that CIs can verify the change.
    → bin/nancyfx-petstore-server.sh
  • Filed the PR against the correct branch: master for non-breaking changes.

Description of the PR

This adds a packageGuid parameter to NancyFXServerCodegen, which allows to pass a fixed GUID, instead of a randomly generated one.

The petstore sample generation script then uses this option (with the last random value from master) to generate the samples. This way there are no random changes between sample updates which obscured the real changes. (This is a part of #5196.)

@ePaul ePaul changed the title Issue #5196: make package GUID nonrandom for nancyfx samples [nancyfx] Issue #5196: make package GUID nonrandom for nancyfx samples Mar 26, 2017
@wing328 wing328 merged commit 9dfbe7e into swagger-api:master Mar 27, 2017
@wing328 wing328 added this to the v2.2.3 milestone Mar 27, 2017
@ePaul ePaul deleted the 5196-make-package-uuid-nonrandom-for-nancyfx-samples branch March 27, 2017 18:16
spr3nk3ls pushed a commit to spr3nk3ls/swagger-codegen that referenced this pull request Mar 28, 2017
…cyfx samples (swagger-api#5209)

* Add packageGuid parameter to NancyFxServerCodegen.

* Use packageGuid parameter for nancyfx petstore samples.
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.

2 participants