<div dir="ltr">Hello!<div><br></div><div>Just submitted a PR for this, <<a href="https://github.com/twisted/twisted/pull/520">https://github.com/twisted/twisted/pull/520</a>>.</div><div><br></div><div>Thanks! :-)</div><div>Eeshan</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jun 22, 2016 at 12:24 AM, Eeshan Garg <span dir="ltr"><<a href="mailto:jerryguitarist@gmail.com" target="_blank">jerryguitarist@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hello again!<div><br></div><div>I am fully aware that it has been a _very_ long time since I mentioned that I would get on with <<a href="https://twistedmatrix.com/trac/ticket/7934" target="_blank">https://twistedmatrix.com/<wbr>trac/ticket/7934</a>>, but I haven't really had the time to actually do anything about it. :-/</div><div><br></div><div>I would love to work on more tickets like <<a href="https://twistedmatrix.com/trac/ticket/7897" target="_blank">https://twistedmatrix.com/<wbr>trac/ticket/7897</a>> and help port Twisted to the new logging system and kind of get back into the flow of submitting patches and contributing to Twisted again (I miss it so much!!).</div><div><br></div><div>My main concerns about moving on with #7934 after my last patch have already been reported by hawkowl in <<a href="https://twistedmatrix.com/trac/ticket/7983" target="_blank">https://twistedmatrix.com/<wbr>trac/ticket/7983</a>>. However, it was a bit unclear to me from the discussions in 7983 as to how to move forward. It would mean a lot to me if someone who is working on this right now (or glyph or hawkowl) could please give me a couple of pointers as to what should be done with the "self.log" issue for porting existing code to the new logging system. </div><div><br></div><div>- Should I declare a log attribute for an entire module to use? </div><div><br></div><div>- Should I rewrite my new patch around hawkowl's suggestion in 7983, use _loggerFor for everything? If so, how do you refer to the Logger instance to compare the log_logger in the event dict with what is expected (since asserting everything in the event dict, including log_logger is crucial for optimum test coverage)? </div><div><br></div><div>- Or is there another way I should be searching for? Has anything changed since 7983 was filed? :-)</div><div><br></div><div>I have already started working on the other suggestions put forth by the reviewers (such as context managers for capturing log events, etc.). Looking forward to contributing more! </div><div><br></div><div>Regards,</div><div>Eeshan Garg</div></div>
</blockquote></div><br></div>