MSS icon indicating copy to clipboard operation
MSS copied to clipboard

rename pages which render in index.py

Open ReimarBauer opened this issue 9 months ago • 3 comments

All functions which do a return render_template(... should become unified with a prefix, e.g. render_help, render_about.

This solves also a naming conflict with the help() and it makes the functions distinguishable from the others there.

ReimarBauer avatar Mar 31 '25 13:03 ReimarBauer

They aren't render functions, they are routes.

matrss avatar Mar 31 '25 13:03 matrss

They aren't render functions, they are routes. Better idea for a prefix?

ReimarBauer avatar Mar 31 '25 14:03 ReimarBauer

A prefix would just add boilerplate for no reason. IMO we can either: rename functions as appropriate according to the route they serve (e.g. change help to mss_help since it serves /mss/help), simply append an underscore to those that do collide with a builtin (a strategy that is pretty common in my experience and even mentioned in pep8), or just keep the ignore as-is.

matrss avatar Mar 31 '25 14:03 matrss