{"push_id":564231380,"size":12,"distinct_size":9,"ref":"refs/heads/master","head":"f0738b887b98b344142285213cfb8332cd65fdd1","before":"1dbdc63f75fc4438b130900ef30749ab0274431c","commits":[{"sha":"8e59460b908086c817a6ac02b777b9ef3b4e69c5","author":{"email":"a737e13205c91e40c7c65a95eda2a9d9c2c34899@gmail.com","name":"Mike Conley"},"message":"Add a clear-pycs alias to .reviewboardrc.\n\nThe .pyc bytecode files have burned us too many times when switching\nbetween revisions of Review Board. No more. This adds a new alias to\nthe Review Board repository so that we can wipe out all pyc files in\none fell swoop.\n\nExample usage:\n\nrbt clear-pycs\n\nTesting done:\n\nRan\nfind . -name \"*.pyc\"\n\nlocally, and saw many pyc files.\nRan\nrbt clear-pycs.\n\nRe-ran\nfind . -name \"*.pyc\"\n\nDidn't see any pyc files listed.\n\nReviewed at https://reviews.reviewboard.org/r/6847/.","distinct":false,"url":"https://api.github.com/repos/chipx86/reviewboard/commits/8e59460b908086c817a6ac02b777b9ef3b4e69c5"},{"sha":"372b4a50de68038758ff417c2f65d7ecff373b2e","author":{"email":"a737e13205c91e40c7c65a95eda2a9d9c2c34899@gmail.com","name":"Mike Conley"},"message":"Merge branch 'release-2.0.x'","distinct":true,"url":"https://api.github.com/repos/chipx86/reviewboard/commits/372b4a50de68038758ff417c2f65d7ecff373b2e"},{"sha":"389f8e2fca79588c4d947b1249581e87cdc547aa","author":{"email":"2314b2e3a4a1f7db165be2aafbf1efd78f28cc97@beanbaginc.com","name":"Christian Hammond"},"message":"Bump to version 2.5 alpha dev.","distinct":true,"url":"https://api.github.com/repos/chipx86/reviewboard/commits/389f8e2fca79588c4d947b1249581e87cdc547aa"},{"sha":"0edd0a8f8e856bf27150f79684b8e58198e97dcd","author":{"email":"c4a40333d2e70b3fcac4e7cd05d0e9037471cc26@gmail.com","name":"David Trowbridge"},"message":"Make development database name dependent on the version.\n\nIt's highly annoying to have to shuffle around databases when you're switching\nbetween major branches. This change updates our prepare-dev.py script to use\ndifferent database names depending on the current VERSION.\n\nTesting done:\n- Used this in my settings_local.py for a long, long time.\n- Blew away my settings_local.py and saw that it was created correctly when\n  running prepare_dev.py\n\nReviewed at https://reviews.reviewboard.org/r/6849/","distinct":false,"url":"https://api.github.com/repos/chipx86/reviewboard/commits/0edd0a8f8e856bf27150f79684b8e58198e97dcd"},{"sha":"a80db85d0efffdaebe98e0c54e68116c5ba7acf9","author":{"email":"c4a40333d2e70b3fcac4e7cd05d0e9037471cc26@gmail.com","name":"David Trowbridge"},"message":"Merge branch 'release-2.0.x'","distinct":true,"url":"https://api.github.com/repos/chipx86/reviewboard/commits/a80db85d0efffdaebe98e0c54e68116c5ba7acf9"},{"sha":"51c20e158b8f796255b0430597729c7672b52586","author":{"email":"c4a40333d2e70b3fcac4e7cd05d0e9037471cc26@gmail.com","name":"David Trowbridge"},"message":"Fix the image review UI comment serialization.\n\nThe generator comment serialization didn't work for the dummy review UI, so I\nchanged it to try to convert to a list. Unfortunately this broke the image\nreview UI, which serializes comments to a dictionary form.\n\nI've changed it so that the basic review UI returns a list rather than an\niterator, and removed the explicit cast.\n\nTesting done:\n- Successfully opened up the image review UI with a comment present.\n- Ran unit tests.\n\nFixes bug 3747.\n\nReviewed at https://reviews.reviewboard.org/r/6852/","distinct":true,"url":"https://api.github.com/repos/chipx86/reviewboard/commits/51c20e158b8f796255b0430597729c7672b52586"},{"sha":"310c3f9e9c80294433ddd2b94ae0ae75c39c2204","author":{"email":"a737e13205c91e40c7c65a95eda2a9d9c2c34899@gmail.com","name":"Mike Conley"},"message":"Fix clear-pycs alias\n\nWe thought this worked. It didn't, due to the double-quotes we'd wrapped\nthe matching pattern in. The switch to double-quotes actually happened\nafter the testing.\n\nThis has now been tested officially without more monkeying around, and\nit successfully clears pycs.","distinct":false,"url":"https://api.github.com/repos/chipx86/reviewboard/commits/310c3f9e9c80294433ddd2b94ae0ae75c39c2204"},{"sha":"7fc06962e7140f4f8c25623a7ccd147b6a41e704","author":{"email":"a737e13205c91e40c7c65a95eda2a9d9c2c34899@gmail.com","name":"Mike Conley"},"message":"Merge branch '2.0.x'","distinct":true,"url":"https://api.github.com/repos/chipx86/reviewboard/commits/7fc06962e7140f4f8c25623a7ccd147b6a41e704"},{"sha":"365cff01ea32dfc436da5bbc79a34e14f8cf2946","author":{"email":"89d07ba9953c2686f150b1ed7057922cd15c5d0d@mit.edu","name":"jessq@mit.edu Qian"},"message":"Clicking username in nav bar now links to user profile instead of user preferences.\n\nWhen you click on your avatar and/or username in the navigation bar, it takes you to the user preferences page. There is already a link to the Preferences page in the user drop down menu, but there isn't a link to your own user page anywhere. Your avatar and username in the navigation bar is now changed to link to your own profile page.\n\nTesting Done:\nClicked my avatar/username from a variety of different places on the site. They all successfully linked to my own profile page.\n\nReviewed at https://reviews.reviewboard.org/r/6865/","distinct":true,"url":"https://api.github.com/repos/chipx86/reviewboard/commits/365cff01ea32dfc436da5bbc79a34e14f8cf2946"},{"sha":"99002b69748656eb07e7f8acda49cc62bcd680a3","author":{"email":"2314b2e3a4a1f7db165be2aafbf1efd78f28cc97@beanbaginc.com","name":"Christian Hammond"},"message":"Add documentation on how to write good descriptions for changes.\n\nA common problem encountered by new contributors to the project is that\nthe commit messages or review request descriptions aren't always\nsufficient to describe the change without requiring additional\ninformation from a bug report or diff. Sometimes they focus on\nreferencing bug numbers or filenames, and sometimes they don't go into\nwhat the core problem was that the change addressed. This makes it\nharder to write release notes or to scan for review requests for a given\ntopic.\n\nThis adds a page to the codebase documentation explaining guidelines and\ntips for writing good review request and commit descriptions. It's\nbroken down into sections for summaries, descriptions, testing, and\ngeneral writing tips.\n\nTesting Done:\nBuilt the docs and viewed them. Formatting looked correct.\n\nRan spell checker. Didn't see any spelling errors.\n\nReviewed at https://reviews.reviewboard.org/r/6858/","distinct":true,"url":"https://api.github.com/repos/chipx86/reviewboard/commits/99002b69748656eb07e7f8acda49cc62bcd680a3"},{"sha":"300dd5bdd1b42fefd649bfceb2a5c69e35923c91","author":{"email":"d48e92e0fe6588b5a3f3112bd87e4d5c011e10af@beanbaginc.com","name":"Barret Rennie"},"message":"Fix blank diff fragments in interdiffs\n\nPreviously, diff fragment comments from interdiffs were sometimes\nresulting in a blank diff fragment. This was because of the diff\nfragment viewer relying on the `'total_line_count'` value in the\nfilediff/interfilediff line counts. However, this value is not always\nguaranteed to have a value. Now we specifically calculate the last\nline in the filediff/interfilediff to avoid this issue.\n\nTesting Done:\nRan unit tests.\n\nVerified that https://reviews.reviewboard.org/r/6815/#comment19874\ndoes not appear as a blank diff with the patch applied. To do this I\nsaved and uploaded all the diffs in order and created an issue on the\nsame lines in the same interdiff.\n\nReviewed at https://reviews.reviewboard.org/r/6838/","distinct":true,"url":"https://api.github.com/repos/chipx86/reviewboard/commits/300dd5bdd1b42fefd649bfceb2a5c69e35923c91"},{"sha":"f0738b887b98b344142285213cfb8332cd65fdd1","author":{"email":"2314b2e3a4a1f7db165be2aafbf1efd78f28cc97@beanbaginc.com","name":"Christian Hammond"},"message":"Refresh some of the information on the codebase docs.\n\nThis fixes up some of the docs to be a bit more modern and to get rid of\nold, misleading information.\n\nThe Contributing Patches documentation has been updated to talk about\nrelease branches and pull requests.\n\nURLs have been updated throughout the docs to use HTTPS instead of HTTP\nwhen applicable.\n\nTesting Done:\nBuilt the docs. Things looked okay.\n\nReviewed at https://reviews.reviewboard.org/r/6863/","distinct":true,"url":"https://api.github.com/repos/chipx86/reviewboard/commits/f0738b887b98b344142285213cfb8332cd65fdd1"}]}