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

Repo status: analyzing...

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.