Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use entity_name event if type come from a string #547

Merged
merged 4 commits into from
Dec 8, 2016

Conversation

frodrigo
Copy link
Contributor

@frodrigo frodrigo commented Dec 6, 2016

No description provided.

@@ -6,6 +6,12 @@

#### Fixes

* [#547](https://github.com/ruby-grape/grape-swagger/pull/547): Use entity_name event if type come from a string - [@frodrigo](https://github.com/frodrigo).

### 0.25.2 (November 30, 2016)
Copy link
Member

Choose a reason for hiding this comment

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

please remove L#11-13 → duplicated

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

@frodrigo frodrigo force-pushed the entity_name_from_string branch from ed7cd22 to cb02a5c Compare December 6, 2016 10:06
@@ -7,6 +7,7 @@
#### Fixes

* [#546](https://github.com/ruby-grape/grape-swagger/pull/546): Move development dependencies to Gemfile - [@olleolleolle](https://github.com/olleolleolle).
* [#547](https://github.com/ruby-grape/grape-swagger/pull/547): Use entity_name event if type come from a string - [@frodrigo](https://github.com/frodrigo).

Copy link
Member

Choose a reason for hiding this comment

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

think you have to add: * Your contribution here. in line 11

Copy link
Member

Choose a reason for hiding this comment

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

The problem with the lines is that they have a period before the - towards the end.

Copy link
Member

Choose a reason for hiding this comment

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

ok @dblock … think now it would be nice to know how it should looks like …
sorry, but yhe syntax of the changelog hasn't my first priority 😉

@LeFnord
Copy link
Member

LeFnord commented Dec 6, 2016

@frodrigo … thanks, please can you make rubocop happy, run it local with bundle exec rubocop

@LeFnord
Copy link
Member

LeFnord commented Dec 8, 2016

yeah … happy to mörge

@LeFnord LeFnord merged commit 385b68f into ruby-grape:master Dec 8, 2016
LeFnord pushed a commit to LeFnord/grape-swagger that referenced this pull request Dec 18, 2016
* Use entity_name event if type come from a string

* Fix CHANGELOG.md format

* Make rubocup happy
LeFnord pushed a commit to LeFnord/grape-swagger that referenced this pull request Feb 9, 2019
* Use entity_name event if type come from a string

* Fix CHANGELOG.md format

* Make rubocup happy
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.

3 participants