Skip to content

Conversation

@AdiSai
Copy link

No description provided.

@AdiSaiAdiSai marked this pull request as draft March 2, 2023 05:26
@kernc
Copy link
Owner

kernc commented Apr 1, 2023

Sorry for the late turnaround. I'm not yet decided, but certainly not strongly opposed to this change. However, in the style of the rest of this library, I'd prefer Strategy.cash being a @property.

Implementing this would prevent people from accessing self._broker._cash as in #493, #340, #124, ...

@kernckerncforce-pushed the master branch 5 times, most recently from 428c361 to 0ce6cabCompareJanuary 21, 2025 07:25
@kernckernc added good first issue Good for newcomers Hacktoberfest https://hacktoberfest.digitalocean.com labels Jan 21, 2025
@kernckerncforce-pushed the master branch 3 times, most recently from 70abc06 to c79ffb0CompareFebruary 17, 2025 01:32
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

good first issueGood for newcomersHacktoberfesthttps://hacktoberfest.digitalocean.com

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@AdiSai@kernc