Kyle McCormick
Kyle McCormick
Taking another look...
I've created a series of PRs based on this branch to make sure that each check fails when a violation is added: * pycodestyle * https://github.com/kdmccormick/edx-platform/pull/21 * pii_check * https://github.com/kdmccormick/edx-platform/pull/18...
@salman2013 I'm currently validating each individual check with this spreadsheet: https://docs.google.com/spreadsheets/d/1KHl4bD9z1OodgLsJLUkobL8R0itMikEOSbHmkIFsMok/edit?gid=0#gid=0
While testing this PR, I found an existing logging issue with our JS CI, and I made a fix that is ready for review: https://github.com/openedx/edx-platform/pull/35954 Once that merges, could you...
@yashathwani this is a really valuable PR, I hope you're still interested in getting it merged. Let me know if there is any way I can help with the CLA...
@yashathwani , do you plan on finishing this PR?
@mphilbrick211 do you know what state this contributor's CLA is in?
Hi @yashathwani , since we have not heard from you, I have opened a new iteration of this PR: https://github.com/openedx/edx-platform/pull/35556
@mphilbrick211 , on his old PR @rediris had said: > I did send a CLA request, but I don't know where I am in the process. I'm going to scrap...
@rediris Don't worry about rebasing. Your change is tiny, so it doesn't matter if it's a few dozen commits out of date.