Skip to content

Fix issue #767, duplicate @JsonProperty#1045

Merged
dilipkrish merged 2 commits intospringfox:masterfrom
mrestivill:master
Nov 16, 2015
Merged

Fix issue #767, duplicate @JsonProperty#1045
dilipkrish merged 2 commits intospringfox:masterfrom
mrestivill:master

Conversation

@mrestivill
Copy link
Copy Markdown
Contributor

There is a duplicated @JsonProperty on the file, could you integrate it? Please

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

apiKey duplicated

@dilipkrish
Copy link
Copy Markdown
Member

@glarfs gaah! thats some copy paste error right there! Thanks for catching that... looks like it has failing tests tho'

@mrestivill
Copy link
Copy Markdown
Contributor Author

The test on my local branch(which is pointing to master) without this commit and it fails on the same point, it's strange. Seems that the code which is failing test is another commit.

@n0mer
Copy link
Copy Markdown
Contributor

n0mer commented Nov 12, 2015

@glarfs @dilipkrish there are definitely 2 failing tests

image

@dilipkrish dilipkrish added the PR label Nov 15, 2015
@dilipkrish dilipkrish added this to the 2.3.0 milestone Nov 15, 2015
@mrestivill
Copy link
Copy Markdown
Contributor Author

There where an issue with tests, Now the tests are fixed

@mrestivill mrestivill closed this Nov 16, 2015
@mrestivill mrestivill reopened this Nov 16, 2015
dilipkrish added a commit that referenced this pull request Nov 16, 2015
@dilipkrish dilipkrish merged commit 62db367 into springfox:master Nov 16, 2015
@dilipkrish
Copy link
Copy Markdown
Member

Thanks! 👍

@n0mer
Copy link
Copy Markdown
Contributor

n0mer commented Nov 16, 2015

@dilipkrish , do you publish snapshot , RC or milestone builds somewhere?

i'm eager to try this :)

@dilipkrish
Copy link
Copy Markdown
Member

A snapshot is automatically published with every build. Details in our documentation on coordinates of the snapshot versions

@dilipkrish
Copy link
Copy Markdown
Member

Actually 2.2.2 was a bad build as regards documentation 😖 The snapshot documentation is probably better

@mrestivill
Copy link
Copy Markdown
Contributor Author

Hi, this fix could be applied also on 2.2.x branch, When are you releasing a new version with this fix?
It would be nice to have an snapshot with this fix as 2.2.3-SNAPSHOT.

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants