U8N WXD
U8N WXD
There's an issue with the backend coverage. Also, this error message isn't very helpful since it doesn't tell you which lines aren't covered: https://github.com/oppia/oppia/runs/7800233372?check_suite_focus=true
(1), (2), and (3) are fixed. We didn't directly fix (6), but we haven't seen it in a while
Sending this to the data handling team since it's not a dev workflow issue
@Manan-Rathi before asking for a full review by all codeowners, please do the following: * Provide proofs of correctness in the PR description * Add a changelog label * Get...
@devarsh10 I've assigned you. See more discussion here: https://github.com/oppia/oppia/pull/15070#discussion_r820423355
@devarsh10 there might not be any files currently where this rule is violated. The goal here is to write a lint check so that we can catch these issues automatically....
@RMeissnerCC we don't use tools like black or prettier because they enforce a different coding style than our [style guide](https://github.com/oppia/oppia/wiki/Coding-style-guide), and as far as I know, no one has been...
@RMeissnerCC that's correct. I think it'd be great to have an automatic formatter that works with Oppia's code style, but we haven't prioritized developing one because once someone gets used...
@RMeissnerCC no, even though we aren't planning on adding an automatic formatter, we still need a lint check to make sure we're indenting correctly
Take a look at https://github.com/oppia/oppia/wiki/Custom-Pylint-checks to learn how to write lint checks for Python files