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

Fix Markdown to adapters documentation #1144

Merged
merged 1 commit into from
Sep 15, 2015

Conversation

bacarini
Copy link
Contributor

From:
selection_009

To:
selection_010

@NullVoxPopuli
Copy link
Contributor

it looks like there is an extra blank row at the top. Is there a way to get rid of that?

@bacarini
Copy link
Contributor Author

There is not @NullVoxPopuli ... It is the header. I tried to fix without it but I couldn't do it.
Any idea on what to add on this header?

@beauby
Copy link
Contributor

beauby commented Sep 13, 2015

Add a normal row, followed by a row of dashes (-----) in order to fill the header.

@bacarini bacarini force-pushed the master branch 2 times, most recently from adf6a7f to 546c4f6 Compare September 14, 2015 00:07
| `ActiveModel::Serializer::Adapter.lookup(name_or_klass)` | The adapter_class, else raises an `ActiveModel::Serializer::Adapter::UnknownAdapter` error |
| `ActiveModel::Serializer::Adapter.adapter_class(adapter)` | delegates to `ActiveModel::Serializer::Adapter.lookup(adapter)` |
| `ActiveModel::Serializer.adapter` | a convenience method for `ActiveModel::Serializer::Adapter.lookup(config.adapter)` |
| Method |Return value|
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you add spaces around "Return value"?

@beauby
Copy link
Contributor

beauby commented Sep 14, 2015

Also, if you could align the middle and right "|", that would be perfect 👍

NullVoxPopuli added a commit that referenced this pull request Sep 15, 2015
Fix Markdown to adapters documentation
@NullVoxPopuli NullVoxPopuli merged commit 736591c into rails-api:master Sep 15, 2015
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