airbnb / epoxy

Epoxy is an Android library for building complex screens in a RecyclerView
https://goo.gl/eIK82p
Apache License 2.0
8.46k stars 730 forks source link

Remove incorrect ksp symbol validation #1323

Closed elihart closed 1 year ago

elihart commented 1 year ago

Latest ksp/xprocessing/kotlin update breaks for us because of this incorrect approach to symbol validation. As far as I can tell it never should have worked because 1) it returns the views as the invalid symbols, not the annotated epoxy model class 2) it doesn't process the valid symbols in that same round, and the valid symbols won't ever be returned for processing in the future

Removing this logic still works fine for our code base and all tests pass, i believe I added this validation optimistically, and it never triggered until now.