You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently TestUser exposes options attribute publicly. There are no clear use cases for me to have this method public. In fact this causes method conflict in TestClient where options is one of http methods actions.
In order to maintain consistency, TestUser can use @options ivar internally instead of accessor or we can rename and protect accessor. This also means deprecating old options accessor.
What do you think?
The text was updated successfully, but these errors were encountered:
Currently
TestUser
exposesoptions
attribute publicly. There are no clear use cases for me to have this method public. In fact this causes method conflict inTestClient
whereoptions
is one of http methods actions.In order to maintain consistency, TestUser can use
@options
ivar internally instead of accessor or we can rename and protect accessor. This also means deprecating oldoptions
accessor.What do you think?
The text was updated successfully, but these errors were encountered: