Andy Seaborne

Results 454 comments of Andy Seaborne

The build is looking good. This ``` [INFO] Running org.apache.jena.ontapi.OntClassMiscTest SLF4J: No SLF4J providers were found. SLF4J: Defaulting to no-operation (NOP) logger implementation SLF4J: See https://www.slf4j.org/codes.html#noProviders for further details. ```...

I'm getting code warnings from Eclipse - not the maven build - here are the non-javadoc ones. Mostly about missing `@Override`: ``` The import java.util.HashSet is never used OntClassMiscTest.java /jena-ontapi/src/test/java/org/apache/jena/ontapi...

Javadoc: less of a concern: there are lots of warnings (291) in Eclipse! Very repetitive and possible better seeing if the HTML output is OK anyway. One class of warning...

The last category of warnings is to do with `@SuppressWarnings` - related to IntelliJ. They can stay. But one is: ``` Unsupported @SuppressWarnings("ALL") TestSpec.java /jena-ontapi/src/test/java/org/apache/jena/ontapi line 66 Java Problem ```...

`test` means the dependency isn't available or necessary for lib users. JUnit is also scope=test. ``` [desktop] >> mvn -Dscope=runtime dependency:tree [INFO] Scanning for projects... [INFO] [INFO] --------------------< org.apache.jena:jena-ontapi >---------------------...

> When generating javadocs using Eclipse I see 100 warnings: I must have some turned if if they are the default. Like IntelliJ, the defaults do rather slide into opinions...

dev@ vote : https://lists.apache.org/thread/pyw05lh55ctkfwnrdmwrxs0kqq7b71o0 result: https://lists.apache.org/thread/gx6s5h00zz5x1d8k5p3g5w6g5x1vg112

And it is now in the SNAPSHOT builds! https://repository.apache.org/content/repositories/snapshots/org/apache/jena/jena-ontapi/ and in the whole system for development builds 5.1.0-SNAPSHOT 2024-05-08T07:49:53 Z and later. This will be the big thing in the...

Documentation can evolve over time. Helping users migrate to the new jena-ontapi is, IMO, the important part. Documentation does not have to be a full tutorial. When the original ontology...

Jena 5.1.0 is in-progress so no rush. This sort of PR is better merged at the beginning of a development cycle rather than at the end 😄