willowtreeapps / assertk

assertions for kotlin inspired by assertj
MIT License
757 stars 84 forks source link

Unify property extractor function names #524

Closed grodin closed 4 months ago

grodin commented 5 months ago

Closes #521

I'm opening this as a draft as the docs aren't updated yet and this is just supposed to be to get the bikeshedding going.

Known things to fix before merge:

evant commented 5 months ago

While we are still bike-shedding the naming I want to call out we'll want to keep but deprecate the old names, not get rid of them entirely.

grodin commented 5 months ago

Yeah, I realised that after I pushed those commits. I was a bit too enthusiastic to get the bike-shedding going I think.

I'll add that to the to-do list above.

grodin commented 4 months ago

Let me know if you want this squashed

grodin commented 4 months ago

I'll let you squash it