Revert "Add missing migration, student course enrollment"

Stanford-Online / edx-platform

The Open edX platform, the software that powers Lagunita!

https://lagunita.stanford.edu

GNU Affero General Public License v3.0

42 stars 18 forks source link

Closed stvstnfrd closed 6 years ago

stvstnfrd commented 6 years ago

This reverts commit 7cd265eede78f5c9a50086776aca2eee593e28bc.

dcadams commented 6 years ago

I assume master was passing previous. How could removing something increase the number of pylint violations?

Number of pylint violations: 3879 Build failed running pavelib.quality.run_pylint: Failed. Too many pylint violations. The limit is 3850.

stvstnfrd commented 6 years ago

Yeah, I'm confused by this too...

I think I actually saw it on an earlier PR, when the migration test was still failing.

Maybe some other code introduced a few violations and we missed it due to the other failures (migration)?

There are a few options; let's chat a bit more f2f