[pyvideo-data] Iceberg PR broken up

will kahn-greene willkg at bluesock.org
Fri May 6 20:40:16 EDT 2016


Easiest way to grab one to review is to comment in the PR saying
something like, "I got this!" Then review it in your leisure and add
comments for any issues you see. I'll keep tabs on things. Some comments
we can spin off into separate issues. Some we might want to tackle in
that PR. Sheila and I can merge any that are good to go.
 
There's a lot of people on this list. If everyone took one PR, we'd get
through the bulk of them pretty quickly. I've done two so far to see
what reviewing them is like. Each took me about ten minutes.
 
Thank you in advance!
 
 
On Thu, May 5, 2016, at 11:49 PM, Paul Logston wrote:
> Hey everybody,
>
> I've broken up the massive PR I issued earlier
> (https://github.com/pyvideo/pyvideo-data/pull/30) with a bunch of
> smaller ones; 73 smaller ones to be exact. One PR for each category
> (ie. pycon-de-2011 or pygotham-2015). Hopefully this makes it easier
> to review.
>
> I've also created a script that will pump the descriptions and
> summaries for a given category out files in a tmp dir. This way, you
> don't have to scan the description and summary lines while they are
> still in the JSON file.
>
> https://gist.github.com/logston/51ab7fbe5edfb8fdf97fb088d09e03a0
>
> Usage flow:
>
> 1. Save gist to local pyvideo-data repo (let's assume you save it to
>    `json_to_rst.py`)
> 2. Check out given branch by category name (eg. `git checkout pycon-de-
>    2011`)
> 3. Run script to populate tmp/ dir with files (eg. `python
>    json_to_rst.py pycon-de-2011`)
> 4. Check the files in the new tmp/ dir found in the pyvideo-data repo.
>    Ensure they contain valid reST.
> 5. Delete the tmp/ dir.
>
> You'd need to re-run steps 2 - 5 for each category you wanted
> to check.
>
> In case you just want to visually check the rendered reST rather than
> the source, here's the first google result for an online renderer:
> http://rst.ninjs.org/ All you would need to do is copy a file's
> contents into the editor section and make sure it looks okay in the
> preview section.
>
> Thanks for any and all help!!
> P
>
>
> --
> Paul Logston
> plog.paullogston.com[1]
> github.com/logston
> _________________________________________________
> pyvideo-data mailing list
> pyvideo-data at python.org
> https://mail.python.org/mailman/listinfo/pyvideo-data
 

Links:

  1. http://plog.paullogston.com/
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.python.org/pipermail/pyvideo-data/attachments/20160506/03f72fb1/attachment.html>


More information about the pyvideo-data mailing list