NightOwl888 commented on PR #961: URL: https://github.com/apache/lucenenet/pull/961#issuecomment-2405416576
> Yeah, it's more than just the RandomExtensions though. TestFramework has types in the Analysis, Codecs, and Search namespaces that conflict with the links on the homepage. Basically, a `xref:Lucene.Net.Search` is finding the test-framework version first always, for some reason. We would have to change all of those types to not conflict, or dig into the docfx source and figure out what the problem is, possibly having to submit a PR to docfx to fix it unless we're doing something wrong as of the latest versions. It seemed to me that just specifying a URL by hand was an easier fix for now. Namespaces are meant to be extended by other assemblies. But for some reason, the default setup of docfx doesn't understand that very well. They apparently made some changes to the way metadata is handled that are not compatible with our config that worked on the old version. But it was never great - in the old version links between subsites wouldn't resolve, so I had to hard code them. Now it seems that links are going to the wrong subsite so they have to be hard coded or painstakingly researched. As sad as it is, hard coding is a solution until we have time to get into the issues with why it fails to resolve many links most of the time. At least now it is spitting out warnings about all of the hard coded URLs so we can easily see what to fix later, so I guess that is an improvement. -- 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
