Git Merge Request #309: add 'continue' lines, change print to logging (merged)

Merging...

Merged

Something went wrong. Please, merge manually

Checking if merge is possible...

Something went wrong. Please, merge manually

Dave Brondsema wants to merge 1 commit from /u/brondsem/allura/ to master, 2018-11-01

This script was missing necessary continue lines within the if not ticket: block. I've fixed that and made a few other minor adjustments.

Commit Date  
[1a2f3f] (db/8081-continue) by Dave Brondsema Dave Brondsema

[#8081] add 'continue' lines, change print to logging

 2018-10-30 20:31:13 Tree

Discussion

  • Shalitha Suranga - 2018-10-31

    Hi.. Dave

    Yeah. logger is better than using print. also continue looks good because we don't want to execute other statements when artifact not found or invalid

     

  • Dave Brondsema

    Dave Brondsema - 2018-10-31

    Thanks. Do you want to go ahead and merge it? :)

     

  • Shalitha Suranga - 2018-11-01
    • Status: open --> merged
     

  • Shalitha Suranga - 2018-11-01

    Merged 👍

     

Log in to post a comment.