MAINTAINERS.txt should use different punctuation around URLs

Oh and this should have tests in the api_link_documentation test and the web test.

Well, it turns out that we are using the core function _filter_url() to turn URLs into links, so the issue is there. I think this is a won't fix.

Yeah, true, good idea. I'd be in favor of removing the <> around the URLs. It doesn't add to readability. Good Novice project (oh, drumm already tagged, good). And I think we should also similarly patch D6 and D7.

The patch looks fine to me. When you upload a patch, you need to set the issue status to "needs review" by the way. :)

anavarre: if you upload a patch for a different version, see the help text under the "attachments" field to find out how to name it to avoid the "needs work" problem above.

Anyway, I'd like to leave this for a day or two to see if anyone objects, then commit this.

It is not "your editor" that is the problem. It is the Drupal 6 URL filter, which is being used to display the URL when the file is displayed on api.drupal.org -- it isn't turning into a link.

Is there really a reason why we shouldn't remove the <> in order to accommodate the limitations of this filter, since at this point there is probably no way it will be improved? Personally, I don't think this punctuation improves the readability of the MAINTAINERS.txt file, and if we need some punctuation, perhaps we can find some alternative punctuation that would keep the readability while still allowing the URL to be displayed as a link on api.drupal.org?

1) We have had .txt files on api.drupal.org for quite a while. Some, such as robots.txt and INSTALL.txt, are arguably part of the Drupal code/documentation base. If MAINTAINERS.txt is not, then why would we include it in the Drupal distribution? We could just maintain it as a page on drupal.org in that case.

2) Not sure historically why are are parsing text files. I've committed a patch to the API module recently that reduces the "parsing" to just turning things into links (e.g., names of files/functions in the Drupal code base, and full URLs).

3) You can find it by typing it into a search.

Thanks! This patch appears to be complete and correct.

I just noticed this very old issue. The patch does not apply now and needs a reroll. Sorry for not getting it committed earlier!

I am not convinced that we really need those email addresses to turn into links in the D6 maintainers file. We should not really have email addresses in there at all... Actually... maybe we should not bother to fix this for Drupal 6 at this point? I'm just going to set this back to 7.x/fixed.