You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@daffodil.apache.org by GitBox <gi...@apache.org> on 2022/02/01 03:17:22 UTC
[GitHub] [daffodil] brnnnfx opened a new pull request #742: Update daffodil-cli link syntax
brnnnfx opened a new pull request #742:
URL: https://github.com/apache/daffodil/pull/742
This is a simple change to
- make the URL appears as a hyperlink to be more consistent with other links
- update the link to HTTPS
--
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: commits-unsubscribe@daffodil.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [daffodil] stevedlawrence commented on a change in pull request #742: Update daffodil-cli link syntax
Posted by GitBox <gi...@apache.org>.
stevedlawrence commented on a change in pull request #742:
URL: https://github.com/apache/daffodil/pull/742#discussion_r796576912
##########
File path: daffodil-cli/README.md
##########
@@ -81,6 +81,6 @@ users@daffodil.apache.org mailing lists. Bugs can be reported via the
Daffodil is licensed under the [Apache License, v2.0](https://www.apache.org/licenses/LICENSE-2.0)
-This product includes The Saxon XSLT and XQuery Processor from
-Saxonica Limited (<http://www.saxonica.com/>), which is licensed under
+This product includes [The Saxon XSLT and XQuery Processor from
+Saxonica Limited](https://www.saxonica.com/), which is licensed under
Review comment:
I found where this text came from in case it was taken from a license or something, in which case we might prefer the exact text. Apparently I came up with this wording in this PR https://github.com/apache/daffodil/pull/502 based on the Saxon licensing page (https://www.saxonica.com/documentation/#!conditions) which says:
> If you produce a product that includes or requires Saxon-HE, please refer to it prominently as "The Saxon XSLT and XQuery Processor from Saxonica Limited", and include the URL of the home page, which is at http://www.saxonica.com/. As a courtesy, please take reasonable steps to ensure that your users know that they are running Saxon.
I think the new text still meets those requirements, and looks a bit better in the REAMDE.
--
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: commits-unsubscribe@daffodil.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [daffodil] mbeckerle merged pull request #742: Update daffodil-cli link syntax
Posted by GitBox <gi...@apache.org>.
mbeckerle merged pull request #742:
URL: https://github.com/apache/daffodil/pull/742
--
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: commits-unsubscribe@daffodil.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org