Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DDFileLogger refactoring #157

Closed
rivera-ernesto opened this issue Nov 7, 2013 · 10 comments
Closed

DDFileLogger refactoring #157

rivera-ernesto opened this issue Nov 7, 2013 · 10 comments

Comments

@rivera-ernesto
Copy link
Member

Seems like many issues need to be solved regarding DDFileLogger (#52, #55, #129, #153 and #155) that would need refactoring instead of simple patches.

Personally I use crash reporters combined with CocoaLumberjack loggers so I may not participate to actively.

Anyone up to the task?

@ratkins
Copy link

ratkins commented Nov 8, 2013

+1

@dvor
Copy link
Member

dvor commented Nov 28, 2013

I'll try to refactor and fix those issues in the nearest future.

@rivera-ernesto
Copy link
Member Author

Nice.

You can make a pull request as soon as you have some commits, not to merge it but to have some feedback/help/cheers.

@dvor
Copy link
Member

dvor commented Dec 1, 2013

It was really hard to reproduce #153 and #155. I spent few hours before I realized that the bug isn't here already. :)

Bug was fixed with #177, those issues can be closed now. @an0 cheers.

@rivera-ernesto
Copy link
Member Author

Yes, when reviewing that pull request I was wondering if it would fix some of those open issues too. But as I don't really log to files I wasn't sure.

Thank you!

@rivera-ernesto
Copy link
Member Author

Thanks @dvor!

@rivera-ernesto
Copy link
Member Author

If everything works fine we should release this by tomorrow as 1.6.5.

@dvor
Copy link
Member

dvor commented Dec 2, 2013

It works fine but it would be better if you guys test it.

@rivera-ernesto
Copy link
Member Author

Sure.

@bpoplauschi
Copy link
Member

@dvor Your contribution is much appreciated.

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

No branches or pull requests

4 participants