The PR contains updates of the module-info.java files. The module names are changed to stem from org.phenopackets.phenopackettools. The version of the phenopacket-schema is bumped to v2.0.2.
There is one thing I am not certain about in the module-info.java of the builder module. Should we require com.google.protobuf in a transitive way? There is already a transitive requirement for org.phenopacket.schema, which requires protobuf. However, the TimestampBuilder includes Timestamp in the API independently of the protobuf, so maybe the transitive requirement is appropriate here too. I'll investigate this further.
The PR contains updates of the
module-info.java
files. The module names are changed to stem fromorg.phenopackets.phenopackettools
. The version of thephenopacket-schema
is bumped tov2.0.2
.There is one thing I am not certain about in the
module-info.java
of the builder module. Should we requirecom.google.protobuf
in a transitive way? There is already a transitive requirement fororg.phenopacket.schema
, which requires protobuf. However, theTimestampBuilder
includesTimestamp
in the API independently of the protobuf, so maybe the transitive requirement is appropriate here too. I'll investigate this further.