New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

refactor: remove testable mangled names #303

Merged

gouline merged 1 commit into master from refactor/remove-testable-mangled-names

Dec 22, 2024

Owner

gouline commented Dec 22, 2024

only internal logic extracted to decrease cyclomatic complexity, not intended to be testable separately, should have mangled names (__ prefixed), to prevent clashing names between mixins
private but separately testable logic should be _ prefixed and sanely named
not prefixed public names should be maintained and refactors count as breaking changes

gouline force-pushed the refactor/remove-testable-mangled-names branch from e26eb7e to 329248c Compare

December 22, 2024 02:08


          refactor: remove testable mangled names

c542a42

gouline force-pushed the refactor/remove-testable-mangled-names branch from 329248c to c542a42 Compare

December 22, 2024 02:13

gouline merged commit c37cf49 into master

1 check passed

gouline deleted the refactor/remove-testable-mangled-names branch

December 22, 2024 02:14

# for free to join this conversation on GitHub. Already have an account? # to comment

Labels

None yet