Skip to content

Conversation

@kevinetienne
Copy link
Contributor

Adding docstrings for view would allow to use the browsable api from the djangorestframework

Choose a reason for hiding this comment

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

Define a `POST` ...

from its using their

Kevin Etienne added 4 commits March 24, 2015 11:07
@kevinetienne kevinetienne force-pushed the add-docstrings-api-views branch from 3a89d64 to a47aa5b Compare March 24, 2015 11:22
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling a47aa5b on add-docstrings-api-views into ed8e647 on master.

Choose a reason for hiding this comment

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

Stray blank line here.

Copy link
Contributor

Choose a reason for hiding this comment

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

Quotes on own line for multi-line docstrings, please

@meshy
Copy link
Contributor

meshy commented Mar 24, 2015

Looks good :)

@kevinetienne
Copy link
Contributor Author

@meshy
Copy link
Contributor

meshy commented Mar 24, 2015

👍

meshy added a commit that referenced this pull request Mar 24, 2015
@meshy meshy merged commit 48376e8 into master Mar 24, 2015
@meshy meshy deleted the add-docstrings-api-views branch March 24, 2015 14:45
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.

5 participants