Skip to content
This repository has been archived by the owner on Aug 7, 2023. It is now read-only.

Fixing error handling from worker. Also accounting for errors finding config. file #169

Merged
merged 1 commit into from
May 17, 2017

Conversation

Xapphire13
Copy link
Contributor

Currently the worker will hang if an error occurs during the finding of the configuration file. This fixes that by moving the try{} higher to encompass it.

Also fixing the error sent from the worker task, this allows the stack to show properly.

@Xapphire13 Xapphire13 added the bug label May 16, 2017
@Xapphire13 Xapphire13 requested a review from Arcanemagus May 16, 2017 02:25
Copy link
Member

@Arcanemagus Arcanemagus left a comment

Choose a reason for hiding this comment

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

That diff makes this look like far larger of a change than it really is lol.

@Xapphire13
Copy link
Contributor Author

Yeah lol, I think I changed like 3 lines. The gift diff algorithm could do with some improvement =]

@Xapphire13 Xapphire13 merged commit 7bf059f into master May 17, 2017
@Xapphire13 Xapphire13 deleted the workerError branch May 25, 2017 19:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants