Skip to content

Error Handling #18

Open
andrei-ivanovici opened this issue Mar 17, 2016 · 4 comments
Open

Error Handling #18

andrei-ivanovici opened this issue Mar 17, 2016 · 4 comments

Comments

@andrei-ivanovici
Copy link
Contributor

We definitely need a better way to log our errors. My first recommendation would be to use a logging framework like Log4Net. I've used it successfully with other projects at work. If we come to an agreement, I can fully integrate it in less than a day

@poma
Copy link
Member

poma commented Mar 17, 2016

Totally agree on that. However I prefer to use NLog. It does the same thing but is simpler and less verbose. Also according to some sources it is faster but it doesn't really matter for this project.

@andrei-ivanovici
Copy link
Contributor Author

Great. I'll pick it up today when I get home. Might create another PR at the end of the day

@poma
Copy link
Member

poma commented Mar 18, 2016

I already integrated it but before adding logging in lots of places I wanted to review your pull request to avoid lots of merge conflicts.

@andrei-ivanovici
Copy link
Contributor Author

cool 👍 , I'll wait for your commits and pick it up afterwards. In the meantime, I'll continue with the MVVM stuff. I can merge any conflicting changes with the mvvm branch

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants