JP

Results 122 comments of JP

Yes. This is a change we'd like to the `DataProvider` API. My current thinking on this is to convert the long list of method arguments into a `RetrieveParams` object: ```...

Not urgent, just an annoying warning in the build pipeline that can't be suppressed. Thanks for looking at this!

The difference between what the comment says and what I actually see in the compiled output is that `_localctx` is _never referenced at all_ in my testing. IOW, it may...

CQL is able to represent a broader of set of types than FHIR (since CQL is not FHIR-specific). The Using CQL with FHIR IG specifies the set of types that...

> I also noticed that when I run this, I get a warning that I'm _not_ seeing on the master branch: > > > > Task :model:compileJava > > /Users/cmoesel/git/cql/pr1413/Src/java/model/build/generated/sources/model/main/java/org/hl7/fhir/Identifier.java:108:...

@cmoesel - The heap pollution was a real issue. The xjc generation was running for all the classes in the schema in the `model` project, not just those relevant to...

@antvaset - We have a goal of >=80% test coverage for new/updated code. Can you add tests for the Evaluators that gets the patch up to 80%?