Skip to content

Raise an AttributeError for __getattr__ methods in AthConfigFlags

Jon Burr requested to merge jburr/athena:ATEAM-898 into master

Closes ATEAM-898.

@amete noted that the existing approach of raising RuntimeError (FlagAddress) or just blindly creating a FlagAddress object (AthConfigFlags) breaks the use of getattr. It also violates basic python rules...

This unifies the logic between the two implementations and ensures that an AttributeError is raised as required by python.

I guess this should also go into 23.0, but I'm not sure how to do that. Is it better to cherry-pick the commit or open an MR to 23.0 and then have it swept to master?

Merge request reports