insightsengineering / teal

Exploratory Web Apps for Analyzing Clinical Trial Data
https://insightsengineering.github.io/teal/
Other
168 stars 33 forks source link

Avoid ID coupling #1160

Open donyunardi opened 4 months ago

donyunardi commented 4 months ago

This introduces a coupling to the ids defined in teal.reporter. If we ever change it for some reason - teal would have to be re-released. The same goes with teal.slice and other packages. I would like to avoid this and keep releases as much independent as possible. This actually happened in the past - on teal.slice refactor we broke tests in teal.modules.hermes just because they were written like this. Please invest in finding a good way to avoid such coupling.

_Originally posted by @pawelru in https://github.com/insightsengineering/teal/pull/1127#discussion_r1524610935_