Show the artifacts URL in job overview
ClosedPublic

Authored by jskladan on Mar 25 2015, 3:05 PM.

Details

Summary

The artifacts URL is now shown in the Job's overview

Test Plan

None so far

Diff Detail

Repository
rEXECDB execdb
Branch
feature/artifacts
Lint
No Linters Available
Unit
No Unit Test Coverage
jskladan retitled this revision from to Show the artifacts URL in job overview.Mar 25 2015, 3:05 PM
jskladan updated this object.
jskladan edited the test plan for this revision. (Show Details)
jskladan added a reviewer: tflink.
tflink accepted this revision.Apr 7 2015, 4:06 PM
This revision is now accepted and ready to land.Apr 7 2015, 4:06 PM
tflink added a comment.May 6 2015, 4:17 PM

@jskladan - can you push this change when you get a chance?

tflink commandeered this revision.May 6 2015, 4:18 PM
tflink edited reviewers, added: jskladan; removed: tflink.

nvm, this was pushed. phab just didn't see the change for some reason. Commandeering and closing

This revision now requires review to proceed.May 6 2015, 4:18 PM
tflink added a comment.May 6 2015, 4:19 PM

nvm about the closing part - looks like I can't do that.

@jskladan - do you have the option to close the revision since you created it?

jskladan commandeered this revision.May 7 2015, 7:16 AM
jskladan edited reviewers, added: tflink; removed: jskladan.

hmm, seems that I can't close it either *sigh*, the only thing I seem to be able to do is abandon it. Should we do that?

kparal accepted this revision.May 7 2015, 12:09 PM
kparal added a reviewer: kparal.
This revision is now accepted and ready to land.May 7 2015, 12:09 PM
jskladan closed this revision.May 7 2015, 12:10 PM

OK, so it needed to be ACKed, in order to be Closed manually.. Thanks @kparal!