<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">https://twistedmatrix.com/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">https://twistedmatrix.com/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">https://twistedmatrix.com/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>