Commits

Author Commit Message Labels Comments Date
Marc-Antoine Ruel
Fix W0613: Unused argument 'FOO'
Andi Albrecht
Silence appstats during tests. See related issue on the App Engine issue tracker: http://code.google.com/p/googleappengine/issues/detail?id=6238 and the comments in the review. Review: http://codereview.appspot.com/5305060
Marc-Antoine Ruel
Adding 'add as reviewer' checkbox to publish form Patch contributed by dominich@google.com http://codereview.appspot.com/5320048/
Branches
chromium
Marc-Antoine Ruel
Fix: E0602: Undefined variable 'FOO'
Marc-Antoine Ruel
Merge default.
Branches
chromium
Marc-Antoine Ruel
Fix E1121: Too many positional arguments for function call
Marc-Antoine Ruel
Fix W0301: Unnecessary semicolon
Marc-Antoine Ruel
Fix W0311: Bad indentation. Found N spaces, expected N
Marc-Antoine Ruel
Fix W0612: Unused variable 'FOO'
Marc-Antoine Ruel
Merge default for now pylint script.
Branches
chromium
Marc-Antoine Ruel
Fix W0611: Unused import FOO
Marc-Antoine Ruel
Fix C0323: Operator not followed by a space
Marc-Antoine Ruel
Fix C0301: Line too long
Marc-Antoine Ruel
Add pylintrc to ease refactoring and maintenance
Marc-Antoine Ruel
Merged default branch into chromium.
Branches
chromium
Andi Albrecht
Add mandatory VERSION variable when updating an instance. Review: http://codereview.appspot.com/5254054/
Andi Albrecht
Add patchset deep links on issues page (fixes issue340). Review: http://codereview.appspot.com/5238050/
anatoly techtonik
{5131052} Update REVISION on 'make serve' r=Andi Albrecht
anatoly techtonik
{5104041} Link to repository issues from issue sidebar (remove empty elements as well) Thanks Andi for review.
Andi Albrecht
Use cleaned_data.get('repo_guid') since this field doesn't exist in IssueBaseForm (fixes issue337). Review: http://codereview.appspot.com/5121046/
Marc-Antoine Ruel
Remove confirmation for commit box with failed try jobs. Effectively backout 54420ac0675a and 032bcc407765. The CQ now runs most necessary tests.
Branches
chromium
anatoly techtonik
Add repository ID field to link Issue with Repository (review 5093045) This fetches ID for Subversion repos, calculates for HG and Git and adds stubs for CVS and Perforce. Also adds the field to search form. Thanks Andi and Marc-Antoine for review and snippets.
Marc-Antoine Ruel
Merge default.
Branches
chromium
Marc-Antoine Ruel
Add created and modified as search filters. Useful to get stats for perf reviews. :) Add a few indexes to help. More indexes may be eventually needed. Review: http://codereview.appspot.com/5082045/
Marc-Antoine Ruel
Merge branch default
Branches
chromium
Marc-Antoine Ruel
Add smart limit for /search interface keys_only requests are super fast. with_messages queries are super slow. Adjust automatic limit accordingly to reduce the likehood of timeout. Review: http://codereview.appspot.com/5078051/
Marc-Antoine Ruel
Add Quick LGTM and LGTM & CQ buttons This permits reviewers to use their mouse to lgtm a change and increase the likehood of CTS. The Quick LGTM and CQ sets the commit bit. Both are not present to the issue owner, the "CQ" flavor is not present if it is already set or if the issue is closed. Add a link to the CQ FAQ to act as a reference point.
Branches
chromium
Marc-Antoine Ruel
Update cpplint and new chromium extra check. Similar to http://codereview.chromium.org/7834045/ but for rietveld.
Branches
chromium
Marc-Antoine Ruel
Remove extra comma, causing the list results to be inside an extra array. Review: http://codereview.appspot.com/5095046/
Marc-Antoine Ruel
Merge default.
Branches
chromium
  1. Prev
  2. Next