Skip to content
This repository has been archived by the owner on Dec 8, 2017. It is now read-only.

Update python docs #23

Merged
merged 2 commits into from Nov 23, 2015
Merged

Update python docs #23

merged 2 commits into from Nov 23, 2015

Conversation

jmcarp
Copy link
Contributor

@jmcarp jmcarp commented Nov 20, 2015

Warning: this is an opinionated patch! I deemphasized doctest (which isn't a testing framework and shouldn't be treated as such) and moved pytest to the top. I also deleted most of the code examples, since they were hard to find--and, more importantly, because many excellent examples already exist, often on the first page of the official docs.

  • Add testing libraries
  • Add testing resources
  • Remove standalone code examples, since they're hard to find; prefer
    linking to existing examples, of which there are many

[Resolves #20]

cc @mbland @arowla

@arowla
Copy link
Contributor

arowla commented Nov 20, 2015

I don't object to jettisoning the examples or the reordering, but I do object to removing my nose tips. 😛 Unless they can be reworked to apply to pytest.... But I have barely ever used pytest, so I wouldn't know.

* Add testing libraries
* Add testing resources
* Remove standalone code examples, since they're hard to find; prefer
  linking to existing examples, of which there are many

[Resolves 18F#20]
@jmcarp
Copy link
Contributor Author

jmcarp commented Nov 20, 2015

@arowla: what do you think about moving the nose tips to a separate file? I was also trying to get the main python page to fit inside a screen or two. If linking to third-party guides won't cut it, I think it would make sense to add nested pages for examples on specific test runners.

@arowla
Copy link
Contributor

arowla commented Nov 20, 2015

👍

On Fri, Nov 20, 2015 at 11:28 AM, Joshua Carp notifications@github.com
wrote:

@arowla https://github.com/arowla: what do you think about moving the
nose tips to a separate file? I was also trying to get the main python page
to fit inside a screen or two. If linking to third-party guides won't cut
it, I think it would make sense to add nested pages for examples on
specific test runners.


Reply to this email directly or view it on GitHub
#23 (comment).

Alison Rowland
Technical Lead, https://fbopen.gsa.gov | 18F
202-317-0124

@jmcarp
Copy link
Contributor Author

jmcarp commented Nov 20, 2015

Updated. Not sure why travis is failing, or even enabled.

@mbland
Copy link
Contributor

mbland commented Nov 20, 2015

Oh, I enabled a bunch of repos before adding .travis.yml files. My bad. I'll start adding them.

@arowla
Copy link
Contributor

arowla commented Nov 23, 2015

Since Travis is just failing on a technicality, going to merge.

arowla added a commit that referenced this pull request Nov 23, 2015
@arowla arowla merged commit 5f014c7 into 18F:18f-pages Nov 23, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants