2020-05-09 UTC
[tantek], indy_, [Katherine], [tw2113] and [chrisaldrich] joined the channel
# 06:21 [tantek] dckc we could use a re-consideration of hAudio in mf2 if you're up for helping with that work strugee, [jgmac1106], Sajesajama_, gRegorLove, Sajesajama, [jeremycherfas], [Rose], [KevinMarks] and [LewisCowles] joined the channel
# 13:16 Zegnat Think I found a way to use the old interoperable whitespacing rules when running the mf test suite for the PHP parser. Lets see if my crazy idea actually works. # 13:25 Loqi [Zegnat] #117 Remove single space from h-entry/urlincontent test [grantcodes] and Henster1 joined the channel
[snarfed], [jeremycherfas] and [jgarber] joined the channel
# 16:56 [jgarber] Zegnat: commented on that microformats/test pull. I _think_ the test suite is actually correct per the spec. # 17:06 Zegnat [jgarber]: I agree with your reading of the spec. I think I am mostly surprised that this is the only test that surfaced that problem with the PHP parser. # 17:06 [jgarber] Maybe that's the only embedded markup test that includes the `<img>` resolution bit? # 17:07 Zegnat Maybe. Would have expected a test specifically for that. But guess not. # 17:07 Zegnat Sadly now I do not know if this is because the PHP parser does something wrong, or beceause I do something wrong when I run the tests :( # 17:10 Zegnat The PHP parser has had different whitespace handling since mid-2018, and I feel like many have been using it without issues since then. Sadly it means the test results are extremely hard to grok # 17:20 Zegnat Adding space to front and back, fail a different test, hahaha # 17:20 [jgarber] Debugging failing tests owing to whitespace errors was def. a challenge. # 17:23 Zegnat What do you think, test wrong this time around? :P # 17:24 Zegnat wants an actual reference parser to test tests now # 17:25 Zegnat Or, hmm, we only add spaces to URLs of course # 17:26 Zegnat Yeah, so the PHP parser has always done this wrong, it seems # 17:26 [jgarber] There are some curious rules in there, but I _think_ the tests are accurate. I just spent a good deal of time with the tests running against some new code so they _should_ be accurate. # 17:27 Zegnat I usually do not pay mind to these whitespace idiosyncrasies because they do not exist in the PHP parser :P # 17:27 Zegnat What with whitespace collapse and all that implemented almost 2 years ago # 17:30 Zegnat Alright, reimplemented use of spaces around img replacements. Only 1 failing test left now. # 17:35 Zegnat All mixed tests passing, 1 datetime issue left on mf2 tests, mf1 tests still a mess. Progress! # 17:40 Zegnat [jgarber]: PR closed, fixed up on the PHP side, thanks for the sanity check! # 17:41 Loqi [jgarber] has 1 karma in this channel over the last year (5 in all channels) # 17:41 Loqi Zegnat has 6 karma in this channel over the last year (56 in all channels) [KevinMarks] joined the channel
# 19:18 Zegnat Did not pass Travis because of the lock file business ... think I fixed it, so lets give it another go. KartikPrabhu, [juju], [chrisaldrich], [tantek], indy, [KevinMarks] and [schmarty] joined the channel
# 21:57 [tantek] GWG, hAudio use-cases are things like when you mention a song and want to refer to a specific song. E.g. in a listen post, or in a play list # 21:58 GWG [tantek]: Is there a video equivalent? # 22:00 [tantek] GWG, there was some work done on hMedia as a replacement for both audio and video gRegorLove joined the channel
# 22:01 GWG [tantek]: Wouldn't that be better to pursue than h-audio? # 22:01 [tantek] in my opinion yes. however it still needs the documentation of use-cases and such KartikPrabhu joined the channel
[snarfed] joined the channel
Zegnat With it php-mf2 only seems to fail 2 tests from the microformats/tests repo’s mf2 tests :D And one looks like a test suite problem: https://github.com/microformats/tests/pull/117
[tantek] GWG, if you want to read the prior work on this, start here: http://microformats.org/wiki/media-info
tantek edited /media-formats (+25) "see also media-info-formats, need more work to split the two" (view diff)