Silvano Cirujano Cuesta
Silvano Cirujano Cuesta
Misbehavior detected on manual test. Better test coverage needed! Moved back to draft until fixed.
@ialarmedalien I'm sorry, but I had to fix a couple of issues that I have detected. I have added the tests for the manual test that has helped me detect...
> @Silvanoc thanks for the conscientious dismissal of the previous review. I'll re-review the PR and make sure we get it merged if @ialarmedalien is unable to get to this...
@ialarmedalien could you please review my latest commits so that this PR can be merged before new changes creates conflicts? Thanks!
@ialarmedalien hopefully I've addressed all your concerns and we are ready to go
@ialarmedalien do you have expertise on warning filters? I'm almost sure that I'm failing to reset the warning registries and since the deprecation warning has been raised somewhere else, I'm...
@alexbrdn could you give this fix a try? I've added the flag `--fix-multivalue-containers` to enable the new context generation behavior, by default disabled (therefore `--no-fix-multivalue-containers`). I've decided to go for...
> @Silvanoc, the fix doesn't quite work as I expected and I don't know why. I have (among others) these two slots in my linkml yamls: > > [...] >...
> @Silvanoc, the fix doesn't quite work as I expected and I don't know why. I have (among others) these two slots in my linkml yamls: > > [...] >...
> So, it looks to me that if a range class doesn't have an identifier, it must get implicitly both `inlined=True` and `inlined_as_list=True` I agree on that. I've extended issue...