#microformats 2018-02-16

2018-02-16 UTC
[kevinmarks], KartikPrabhu, [eddie], tantek, [shurcool], [prtksxna], barpthewire, gRegorLove_, nitot, [gerwitz], nitot_, MeanderingCode, [cleverdevil] and vivus joined the channel
#
Zegnat It looks like tantek, aaronpk, and I are all in agreement on the p-name resolution. See last proposal in this comment: https://github.com/microformats/microformats2-parsing/issues/6#issuecomment-366298739
#
Zegnat Would be nice to get the thumbs up from some actual parser implementers though: gRegorLove_. KartikPrabhu (if you want to take up Python).
#
Zegnat Lets see if we can get this issue closed and in the spec!
#
Loqi [Zegnat] Documenting from yesterday’s chat, because nobody could remember this and I can’t find it elsewhere: * @tantek would like to [keep parser changes conservative](https://chat.indieweb.org/dev/2018-02-15/1518732086447100) but [is “amenable to e...
#
KartikPrabhu Zegnat: will take a look this weekend and see
#
aaronpk this is going to make my life so much easier
[kevinmarks] joined the channel
#
[kevinmarks] I would like to move mf2py into microformats - what would be the best way to integrate it with the tests repo? Last time I spent a chunk of time on this being bale to iterate on the tests and the parser I'm parallel was really handy
#
tantek [kevinmarks]: I think you have to get tommorris to move it since his mf2py repo has the issues and we don't want to lose those
#
[kevinmarks] Unless we can pesatas them other bridgy
#
[kevinmarks] ;)
#
KartikPrabhu is very bad at tests
[eddie] joined the channel
#
gRegorLove Re: php-mf2 semver, this merged commit (still unreleased) is a breaking change, so it would probably be good to get the implied p-name in the same release since it's also breaking. I'll work on it soon.
#
gRegorLove https://github.com/indieweb/php-mf2/pull/125
#
Loqi [gRegorLove] #125 Add rel-urls object
#
gRegorLove For clarity, are there any objections to how Zegnat stated it here: https://github.com/microformats/microformats2-parsing/issues/6#issuecomment-366298739
#
Loqi [Zegnat] Documenting from yesterday’s chat, because nobody could remember this and I can’t find it elsewhere: * @tantek would like to [keep parser changes conservative](https://chat.indieweb.org/dev/2018-02-15/1518732086447100) but [is “amenable to e...
#
gRegorLove If not I'll do that, otherwise I'll do the more conservative option.
#
tantek huh? I thought that was the more conservative option
#
gRegorLove I interpret the original as the more conservative option: https://github.com/microformats/microformats2-parsing/issues/6#issuecomment-355632808
#
Loqi [tantek] Restating for clarity, and adding children as another way to stop implying p-name "p-name MUST NOT be implied if there are any explicit p-* properties or any nested microformats" Need consensus positive feedback from parser developers and one p...
#
gRegorLove The other one is where you're "amenable" to expanding to e-*
#
tantek gRegorLove: then what was confusing was that that makes 3 options then AFAIK
#
tantek so "more" is unclear without specifics
#
tantek "more conservative" that is
#
gRegorLove Now I'm getting confused, heh.
#
gRegorLove The original proposal is "p-name MUST NOT be implied if there are any explicit p-* properties or any nested microformats"
#
gRegorLove A newer alternative is "p-name MUST NOT be implied if there are any explicit p-* or e-* properties, or any nested microformats."
#
gRegorLove So if no objections to the newer alternative, I'll work on that in php-mf2.
#
tantek and Zegnat's original proposal, p-name MUST NOT be implied if there are any explicit properties
#
gRegorLove Aha
#
gRegorLove I thought your objection to Zegnat's original proposal was the preference for more conservative parser changes, so I didn't think it was an option currently.
#
gRegorLove Zegnat on gh: "I have not been able to document any cases where more than this is necessary either.
#
gRegorLove "
#
gRegorLove "this" being the newer alternative.
[cleverdevil] joined the channel
#
Zegnat I originally proposed to not imply p-name when any properties were given.
#
Zegnat tantek amended his proposed solution to p- and e-, opting to be more conservative than my proposal.
#
Zegnat I then confirmed that tantek’s amended proposal (the “this”) would indeed fix all my documented problem cases, so I had no complaints.
#
Zegnat Hope that clears it up :)
#
Zegnat Currently voting on the proposal as iterated in aaronpk’s latest comment: “the current proposal of having p-* and e-* and h-* stop the implied name” https://github.com/microformats/microformats2-parsing/issues/6#issuecomment-366308533
#
Loqi [aaronpk] Nope, wasn't intending to hold things up, just wanted to put that there for the record. I agree with the current proposal of having `p-*` and `e-*` and `h-*` stop the implied name.
chrisaldrich joined the channel
#
gRegorLove +1'd
barpthewire, eddie, tantek, [kevinmarks], [eddie], [cleverdevil], KartikPrabhu, gRegorLove and [mifga] joined the channel