Skip to content

Improved debugging features #23

Merged
jonnypress merged 3 commits intomasterfrom
aidans-branch
Jan 6, 2017
Merged

Improved debugging features #23
jonnypress merged 3 commits intomasterfrom
aidans-branch

Conversation

@aidanogorman
Copy link
Copy Markdown
Contributor

@aidanogorman aidanogorman commented Jan 5, 2017

Improved debugging features in the loadf function so that errors inside files are no longer masked. If an error occurs when files are loaded in, it will flag at that point in the file and enter debug mode.

I'm happy in my eyes that this change won't have any other implications and there are no other error traps that will cause issue.

@aidanogorman aidanogorman requested review from AllanMoore20, aasclarke, jonnypress and picoDoc and removed request for AllanMoore20 and aasclarke January 5, 2017 17:46
Copy link
Copy Markdown
Contributor

@jonnypress jonnypress left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you make that an if-else rather than a double if? Seems like it would be a bit neater.

@picoDoc
Copy link
Copy Markdown
Collaborator

picoDoc commented Jan 6, 2017

looks good to me

@jonnypress jonnypress merged commit a482daf into master Jan 6, 2017
@jonnypress jonnypress deleted the aidans-branch branch January 6, 2017 12:50
FlyingOE pushed a commit to FlyingOE/TorQ that referenced this pull request Mar 10, 2017
* 'master' of https://github.com/AquaQAnalytics/TorQ: (95 commits)
  Fixed issue where eod would fail when wdb was set to partbyattr and t… (DataIntellectTech#42)
  Mergereplay (DataIntellectTech#41)
  Preventing tabsizes appearing in hdb (DataIntellectTech#39)
  Subcut (DataIntellectTech#38)
  Added fix to .email.senddefault to terminate if no connection to server (DataIntellectTech#37)
  added passwords dictionary for connecting to non torq processes and modified opencon function (DataIntellectTech#36)
  Blog page and short descriptions of previously unmentioned files (DataIntellectTech#35)
  Documentation kdbappcode (DataIntellectTech#34)
  updated to 3.0.0
  added ldap permissioning
  Ldap (DataIntellectTech#29)
  eodtime bugfix, improved error catching at rollover (DataIntellectTech#31)
  Update utilities.md with comments on API table generation (DataIntellectTech#33)
  Docsite (DataIntellectTech#32)
  Delete Documentation pdf (DataIntellectTech#30)
  Readme update (DataIntellectTech#28)
  Mkdocs site details (DataIntellectTech#27)
  Added permissions.q to the documentation (DataIntellectTech#25)
  Permissions (DataIntellectTech#24)
  Improved debugging features  (DataIntellectTech#23)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants