NightOwl888 commented on PR #1086:
URL: https://github.com/apache/lucenenet/pull/1086#issuecomment-2568724055

   > I'll add in the failure message to print the class name.
   > 
   > This fails currently on ADO, wrong path to the runsettings file. I'll 
replace this with command-line args which will solve the problem. I think it's 
worthwhile to have it in the test log output and trx file too, not just in the 
failure message.
   
   Fair enough. I would like to see what the UX looks like without the extra 
info in the log file, though. Could we make this a parameter that defaults to 
enabled if not supplied in ADO that allows it to be disabled in a specific ADO 
pipeline? For example, the properties that are [documented at the top of the 
`azure-pipelines.yml` 
file](https://github.com/apache/lucenenet/blob/dbc602c7f872189516dc132045f1cc2b69d5ca7e/azure-pipelines.yml#L37-L52)
 can all be defined in the pipeline. Perhaps name it `DisplayFullName` and put 
it under the "Testing variables".


-- 
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: dev-unsubscr...@lucenenet.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to