Alexandre Terrasa
Alexandre Terrasa
That's one of the problem with `is_borken`, you never know what you can do with them before an assert fails. Exceptions could be useful here!
@vinistock I believe this PR is ready. I'll leave you with the final word of merging or closing 👍
> in the code base And the gems I guess?
This usage of `T.bind` is strange. What would happen if I included the module in another class? ```rb class D include M end ``` I _think_ what you're looking for...
I like the idea of the another executable. I wonder if we could integrate a kind of manual step to run each PR through this tool before merging them rather...
> many syntax examples we aren't testing yet We should add these to the test fixtures 🐛
Hey Jack, It's in our plans to add support for `requires_ancestor { T.class_of(AbstractObject) }`. I'll keep this issue open as a reminder. Thanks for the report.
Closing as this as been fixed more elegantly in #6589.
Closing this issue in favor of a more general solution: https://github.com/Shopify/tapioca/issues/1361.
> @Morriar you and Ufuk have a branch somewhere with a prototype for this, right? If so, can you link it here? It's very WIP but it's here: https://github.com/Shopify/tapioca/commit/e6e1dba26a3a63c2c33c71732f6073da00eb05d6.