Updated XML parser #7312

Closed
marijnh wants to merge 3 commits from github/fork/deebugger/master into master
marijnh commented 2011-07-26 14:35:44 +02:00 (Migrated from gitlab.com)

Not sure why the pull request was closed (by me, for some reason), but this is a newer one, with better handling of XML declaration and attributes.

Not sure why the pull request was closed (by me, for some reason), but this is a newer one, with better handling of XML declaration and attributes.
marijnh commented 2011-08-12 16:26:44 +02:00 (Migrated from gitlab.com)

Hey. You may be wondering why this was left unanswered for so long. There are two reasons. First, I'm an unorganized slob. Second, your mode seems to be unfinished, and doesn't handle HTML anymore. Thus, it is not viable to simply merge it into the main repository. If you polish it up further, and move it into a new mode/ subdirectory, I'll gladly take a closer look.

Hey. You may be wondering why this was left unanswered for so long. There are two reasons. First, I'm an unorganized slob. Second, your mode seems to be unfinished, and doesn't handle HTML anymore. Thus, it is not viable to simply merge it into the main repository. If you polish it up further, and move it into a new mode/ subdirectory, I'll gladly take a closer look.
marijnh (Migrated from gitlab.com) closed this pull request 2011-08-12 16:26:44 +02:00

Pull request closed

Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
codemirror/codemirror5!7312
No description provided.