Re: Pod::Simple doesn't warn when the text of a definition =item matches /[\*\d]/; and a fix to this bug
On Jan 18, 2013, at 10:09 AM, Ricardo Signes wrote: > I believe it could be bundled, and would be worth the hassle. Maybe. :-) Well, if it's a hassle someone else wants, great! :-) David
Re: Pod::Simple doesn't warn when the text of a definition =item matches /[\*\d]/; and a fix to this bug
On 01/18/2013 10:33 AM, David E. Wheeler wrote: On Jan 18, 2013, at 7:31 AM, Ricardo Signes wrote: I hope we can agree that if the test results bit gets sorted out, we're in favor of this warning..? I have submitted https://github.com/theory/pod-simple/pull/44 +1. Will merge it if there are no objections. +1
Re: Pod::Simple doesn't warn when the text of a definition =item matches /[\*\d]/; and a fix to this bug
* "David E. Wheeler" [2013-01-18T12:33:52] > Good idea, though it will require that older versions of Perl install a JSON > parser just to run the tests… I believe it could be bundled, and would be worth the hassle. Maybe. :-) -- rjbs signature.asc Description: Digital signature
Re: Pod::Simple doesn't warn when the text of a definition =item matches /[\*\d]/; and a fix to this bug
On Jan 18, 2013, at 7:31 AM, Ricardo Signes wrote: >> I hope we can agree that if the test results bit gets sorted out, we're in >> favor of this warning..? > > I have submitted https://github.com/theory/pod-simple/pull/44 +1. Will merge it if there are no objections. > I also had the idea, mentioned in the commits, that we could consider > replacing > the XML output for testing with JSON output. This would make it easier to > test > subsections and identify structural differences, since we have a JSON parser > in > core but not an XML parser. Good idea, though it will require that older versions of Perl install a JSON parser just to run the tests… David
Re: Pod::Simple doesn't warn when the text of a definition =item matches /[\*\d]/; and a fix to this bug
* Ricardo Signes [2013-01-18T08:54:05] > I hope we can agree that if the test results bit gets sorted out, we're in > favor of this warning..? I have submitted https://github.com/theory/pod-simple/pull/44 I also had the idea, mentioned in the commits, that we could consider replacing the XML output for testing with JSON output. This would make it easier to test subsections and identify structural differences, since we have a JSON parser in core but not an XML parser. -- rjbs signature.asc Description: Digital signature
Re: Pod::Simple doesn't warn when the text of a definition =item matches /[\*\d]/; and a fix to this bug
* Marc Green [2012-02-19T16:19:08] > When given the following input, Pod::Simple does not warn that the [text] > portion of the 2nd and 3rd =item is invalid. > > =over > =item a definition > some text > =item * > a bullet > =item 1 > a number > =back > > [perlpodspec cited] > > I am not sure why it doesn't warn in this situation, but does warn for > other =item type mismatches. This seemed to go nowhere, but was brought up to help get Pod::Checker using Pod::Simple. I hope we can agree that if the test results bit gets sorted out, we're in favor of this warning..? -- rjbs signature.asc Description: Digital signature