afs commented on PR #3629:
URL: https://github.com/apache/jena/pull/3629#issuecomment-3636442032

   Hi @AtesComp - good to see this PR!
   
   I downloaded the PR and ran the Jena build (`mvn clean install`) which came 
across two issues:
   
   1.The file 
`jena-fuseki2/jena-fuseki-ui/src/components/dataset/CodeMirror.vue` needs a 
license header comment (see other files in the same directory).
   
   2. The yarn.lock file is not in-step with packages.json. 
   
   With workarounds for those two points, the build+tests works.
   
   Also - please could you rebase the code to the latest state of `main` 
because today is our "javascript dependencies update day" from dependabot and 
today's batch changed `yarn.lock`.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to