uranusjr commented on pull request #16634: URL: https://github.com/apache/airflow/pull/16634#issuecomment-921556087
> Can I just create a PR to do it or does it need to be done at a specific time? No specific timeline, this is just something that I want to do. So feel free to do a PR anytime you feel fit. The easy part would be to split helpers into a `views/utils.py` file, and some small view classes into their own files. The most difficult part would be to split up a giant `Airflow` class that spans >2k lines itself. It has a ton of not-really-related views, but we can’t just split it since the class name tied to URL back-references. There are a number of ways to work around that (mixins, composition, Flask URL registration hacks, or just changing the backrefs, etc.) and I’m not sure what’s the best technique. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
