-
-
Notifications
You must be signed in to change notification settings - Fork 43
General review, follow-on to mavenized documentation #342 #345
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
Conversation
|
I found a few URLs that still have problems: [
{
"404-errors": [
{"backup.xml": "http://www.quartz-scheduler.org/documentation/quartz-2.x/tutorials/tutorial-lesson-06.html"},
{"beginners-guide-to-xrx-v4.xml": "http://localhost:8080/exist/apps/term/index.html"},
{"scheduler.xml": "http://www.quartz-scheduler.org/documentation/quartz-2.x/tutorials/crontrigger.html"},
{
"devguide_indexes.xml": [
"http://www.ordnancesurvey.co.uk/oswebsite/products/try-now/sample-data.html",
"http://www.vividsolutions.com/jts/bin/JTS%20Technical%20Specs.pdf",
"http://www.ordnancesurvey.co.uk/products/osmastermap/layers/topography/sampledata/2182-SS7886-2c1.gz",
"http://www.vividsolutions.com/jts/jtshome.htm"
]
}
]
},
{
"other-error": [
{
"deployment.xml": {
"error": "titanium.dstc.edu.au: nodename nor servname provided, or not known",
"href": "http://titanium.dstc.edu.au"
}
},
{
"xforms.xml": [
{
"error": "Connect to www.betterform.de:80 [www.betterform.de/212.204.63.100] failed: Operation timed out (Connection timed out)",
"href": "http://www.betterform.de"
},
{
"error": "Connect to betterform.de:80 [betterform.de/212.204.63.100] failed: Operation timed out (Connection timed out)",
"href": "http://betterform.de/trac"
},
{
"error": "Connect to betterform.de:80 [betterform.de/212.204.63.100] failed: Operation timed out (Connection timed out)",
"href": "http://betterform.de"
},
{
"error": "Connect to betterform.de:80 [betterform.de/212.204.63.100] failed: Operation timed out (Connection timed out)",
"href": "http://betterform.de/reports/Firefox-3.0-Mac-OS-X-10.6-ConformanceReport1.1.html",
"doc": "xforms.xml"
}
]
},
{
"devguide_indexes.xml": [
{
"error": "geotools.codehaus.org: nodename nor servname provided, or not known",
"href": "http://geotools.codehaus.org/"
},
{
"error": "lists.refractions.net: nodename nor servname provided, or not known",
"href": "http://lists.refractions.net/m2/org/geotools/gt2-epsg-hsql/"
},
{
"error": "javadoc.geotools.fr: nodename nor servname provided, or not known",
"href": "http://javadoc.geotools.fr/snapshot/org/geotools/referencing/factory/epsg/FactoryOnHSQL.html"
}
]
}
]
}
]Suggestions welcome! |
duncdrum
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Joe these are very helpful. Since these are based of the maven changes, these can only go into v5 of the docs, however we have at least one more v4 release scheduled.
I'd be happy to merge a cherry-picked branch of just the non-maven stuff preferably after #343 is in. Where we could then also add a link checker to the XQsuite tests.
|
@duncdrum Great to hear. Sure, cherry pick away. |
- Move Author Reference to other documentation administration - Correct link to CONTRIBUTIING.md
- Grammar - Reduce phrases like “This article will tell you how to…” or “Provides you with information about how to” to “How to…”
From author guidelines: “use the target's document name (without the .xml extension) as the link address.”
and miscellaneous catches
|
Rebased against |
|
lets fix the failing CI first |
|
@duncdrum Thanks for the nudge - I forgot to run |
DO NOT MERGE until #342 is merged. This is actually based off of that branch.Update (2018-05-22): #342 was superseded by #349 (the
5.0.0-RC1branch), so I rebased against it instead.