Klemen
Klemen
 Right now we are in release phase so what we need is testing of master branch, not reviews of feature branches
I will try to review it after I finish the release ;)
> Can we be sure TabEntry is not null? I think unless some converter from old workspaces is not implemented incorrectly, then yes, it should not be null per-se
You could probably remove this annotation
@Defeatomizer, I stand corrected, tabEntry can be null (I went through some workspaces and found some null cases) so those checks need to be reintroduced. If the converter in #4399...
So far it looks ok to me, I just need to get a timeslot to do some more proper testing. In the meantime, I will wait for any community feedback...
Example of such report: https://github.com/MCreator/MCreator/actions/runs/8957007479 Example of >2 deep nesting: https://github.com/MCreator/MCreator/actions/runs/8957007479/job/24599538540#step:5:1303
Ok, I will test it when I get time to look into it. Thanks!
https://github.com/MCreator/MCreator/actions/runs/9616678764 Does not seem to fix. In our case it seems `{{BREAD_CRUMB}}` is empty based on linked report Config: https://github.com/MCreator/MCreator/pull/4901/files#diff-faff1af3d8ff408964a57b2e475f69a6b7c7b71c9978cccc8f471798caac2c88
Aha, I see, the problem is on Gradle level because it is not even including information about this in the XML. No need to attach XML as this is not...