rvesse commented on PR #1989: URL: https://github.com/apache/jena/pull/1989#issuecomment-1680200251
Could you please describe the bugs this PR is fixing? I can't see what those are from reading the PR diff. There's a lot of whitespace change in this PR as well which makes understanding what's changed very difficult even with Hide Whitespace mode enabled so describing what's actually changed would be useful. From what I can follow you are changing the output format which I don't think we'd want to do because that potentially breaks all existing users of this tool. The current output format is intentionally following classic `diff` style. Perhaps consider putting the alternative output format behind a new flag/option so we retain backwards compatibility while allowing users like yourself to opt into a newer output format? -- 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]
