Skip to content

Fix missing return statement #1440

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Oct 19, 2017
Merged

Fix missing return statement #1440

merged 1 commit into from
Oct 19, 2017

Conversation

meyerbaptiste
Copy link
Member

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets N/A
License MIT
Doc PR N/A

@meyerbaptiste meyerbaptiste force-pushed the fix_cs_missing_return_statement branch 7 times, most recently from e42a82d to 21f7b52 Compare October 19, 2017 08:56
@meyerbaptiste meyerbaptiste force-pushed the fix_cs_missing_return_statement branch from 21f7b52 to 11cb9c6 Compare October 19, 2017 08:57
@meyerbaptiste meyerbaptiste changed the title Fix CS & missing return statement Fix missing return statement Oct 19, 2017
@dunglas dunglas merged commit 6a5c9ab into api-platform:2.0 Oct 19, 2017
@dunglas
Copy link
Member

dunglas commented Oct 19, 2017

Thanks @meyerbaptiste

@meyerbaptiste meyerbaptiste deleted the fix_cs_missing_return_statement branch October 19, 2017 13:37
hoangnd25 pushed a commit to hoangnd25/core that referenced this pull request Feb 23, 2018
@dunglas
Merge pull request api-platform#1440 from meyerbaptiste/fix_cs_missin…
28ea37f 
…g_return_statement

Fix missing return statement
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
[RFR] Ready for Review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@meyerbaptiste @dunglas