#microformats 2020-05-02

2020-05-02 UTC
jgmac1106 and [Aaron_Klemm] joined the channel
#
[tantek] in general feel free to link to PRs here and @-me to review and I'll do the best I can
#
[tantek] I may not understand all the code, and I may request additional reviewers accordingly
jgmac1106, [fluffy], [snarfed], cm2, gRegorLove and mauz555 joined the channel
#
Zegnat To step away from conflating handling invalid URLs and defining what normalisation we expect, I propose splitting up the two tickets: https://github.com/microformats/microformats2-parsing/issues/9#issuecomment-622887901
#
Zegnat Anyone see any reasons why we may want to keep it as one?
#
Loqi [Zegnat] Per https://github.com/microformats/tests/pull/112#pullrequestreview-404489613: should we split out normalisation to a different ticket and focus only on invalid URLs here? It sounds like everyone’s preferred behaviour would be to pass on the va...
#
Zegnat works through some php-mf2 PRs in the mean-time
jgmac1106 joined the channel
#
Zegnat When you can run tests, it becomes a lot easier to do PR reviews. Sadly it takes some time to set them up locally to actually run on branches that do not have them :(
#
Zegnat gRegorLove, GWG, thanks both for all the code and reviewing work! Merged two more PRs.
[LewisCowles], tom, [snarfed], [pfefferle], mauz555_, mauz555, Sajesajama, Sajesajama_, Sajesajama__, [tantek], gRegorLove, eightfold1 and [tim038] joined the channel
#
Zegnat GWG: I may have figured out why Travis wasn’t working even with `composer update`. Updated the ticket, but really need someone with more Travis knowledge to verify.
#
GWG Which is not me
#
Zegnat I guess aaronpk added Travis 5 years ago, and then added the HTML5 parser 2 years ago, so he is the person to ask? :P Else we can just make a commit with my try.
#
Zegnat But will have to wait for the morrow. Sleep time for me :) Thanks for all the hard work, again!