Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

various corrections #411

Merged
merged 1 commit into from
Sep 13, 2023
Merged

various corrections #411

merged 1 commit into from
Sep 13, 2023

Conversation

bbannon
Copy link
Contributor

@bbannon bbannon commented Sep 13, 2023

correct spellings
trim a few lines
remove a few extra spaces
correct bad links that I could figure out
shrink a few code snippets for efficiency

@bbannon
Copy link
Contributor Author

bbannon commented Sep 13, 2023 via email

@craigk5n
Copy link
Owner

Why? They both go to the same page. And there are very few links that require the 'www.' any more.

On Wed, Sep 13, 2023 at 10:16 AM Craig Knudsen @.> wrote: @.* commented on this pull request. ------------------------------ In tests/phpunit.xml <#411 (comment)>: > @@ -1,5 +1,5 @@ - We actually need to leave this as "www.w3.org". — Reply to this email directly, view it on GitHub <#411 (review)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHJ7GLVLM7AABAU5E5UWSRLX2HL5VANCNFSM6AAAAAA4WSQKZY . You are receiving this because you authored the thread.Message ID: @.***>

Here's how ChatGPT answered this question:

The www.w3.org is the official domain for the World Wide Web Consortium (W3C). When it comes to XML namespaces and schema locations, it's crucial to use the exact URI provided by the standard or definition, as XML parsers may rely on these exact URIs to understand or validate the document.

In the given XML header, http://www.w3.org/2001/XMLSchema-instance refers to the XML Schema instance namespace. Changing this URI (even if the changed URL still points to the same resource) can lead to compatibility and validation issues.

In other contexts, omitting "www" from a URL might be okay if the server is configured to handle requests to the domain without the "www" prefix in the same way. But in the context of XML namespaces and schema URIs, it's important to preserve them exactly as defined.

In short, for the sake of XML compatibility and validation, you should not drop the "www." from the http://www.w3.org/2001/XMLSchema-instance namespace.

@bbannon
Copy link
Contributor Author

bbannon commented Sep 13, 2023 via email

@bbannon
Copy link
Contributor Author

bbannon commented Sep 13, 2023 via email

@craigk5n
Copy link
Owner

It's just the one file. I'll go ahead and merge and fix it on my end.

@craigk5n craigk5n merged commit 4da50d1 into craigk5n:master Sep 13, 2023
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants