History | Log In     View a printable version of the current page.  
Issue Details (XML | Word | Printable)

Key: QB-2369
Type: Improvement Improvement
Status: Resolved Resolved
Resolution: Fixed
Priority: Major Major
Assignee: Steve Luo
Reporter: John Szakmeister
Votes: 1
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
QuickBuild

Fill in the context for the GitHub status API...

Created: 14/Mar/15 02:55 PM   Updated: 16/Aug/16 01:18 PM
Component/s: None
Affects Version/s: 6.0.9
Fix Version/s: 6.1.22

Original Estimate: Unknown Remaining Estimate: Unknown Time Spent: Unknown
File Attachments: None
Image Attachments:

1. pull-request-status.png
(156 kb)


 Description  « Hide
At the moment, when QuickBuild posts its status to GitHub, it's not filling in the context variable which results in a label of 'default' being applied to the status. It'd be nice to fill this in with something like "continuous-integration/quickbuild" or "continuous-integration/quickbuild/user-defined-string"). See the context parameter in https://developer.github.com/v3/repos/statuses/#parameters for more information.

BTW, sorry for all the tickets lately. I don't expect you to fix them all or fix them all right away. But I wanted to share the things I've been running across.

 All   Comments   Work Log   Change History      Sort Order:
Change by Robin Shen [09/Jul/16 12:08 AM]
Field Original Value New Value
Assignee Robin Shen [ robinshine ] Steve Luo [ steve ]

Change by Steve Luo [14/Jul/16 01:21 AM]
Fix Version/s 6.1.20 [ 11675 ]

Change by John Szakmeister [14/Jul/16 10:10 AM]
Attachment pull-request-status.png [ 10576 ]

Change by Robin Shen [15/Aug/16 10:06 PM]

Change by Steve Luo [16/Aug/16 01:18 PM]
Status Reopened [ 4 ] Resolved [ 5 ]
Fix Version/s 6.1.22 [ 11677 ]
Fix Version/s 6.1.20 [ 11675 ]
Resolution Fixed [ 1 ]