#dev 2020-05-30
2020-05-30 UTC
flex14, KartikPrabhu, [chrisaldrich], geoffo, gRegorLove, [John_Payne] and test joined the channel
[prtksxna] and KartikPrabhu joined the channel
[tw2113], nickodd, [jeremycherfas], KartikPrabhu, dckc, deathrow1, moppy and dogfarted joined the channel
# Zegnat !tell [LewisCowles] when you are ready for round two of what-the-huh-DOMDocument, my initial testing says almost *any* parsing option will fix the behaviour: https://gist.github.com/Zegnat/a94489e9b7d5501193e724e336bc6052#file-lookingforoptionsresults-txt
[grantcodes] joined the channel
[prtksxna] and dogfarted joined the channel
# @rMdes_ Can't wait for Known to have a "reader" built-in so I can use it with Indigenous micropub/microsub app #indieweb (https://blog.rmendes.net/s/zEdP8) (twitter.com/_/status/1266708427625172995)
flex14 and jeremych_ joined the channel
# @nickang ↩️ Oh there's an actual web service for this kind of thing → https://webmention.io/ (twitter.com/_/status/1266711386098368513)
[schmarty] joined the channel
# Loqi h-feed is a microformats2 draft specification with a top level feed object to contain root class(h-*) objects, usually h-entry posts and optionally a common author, name, and representative photo https://indieweb.org/h-feed
flex14 joined the channel
dogfart joined the channel
[snarfed] joined the channel
# [snarfed] vilhalmer i assume you've seen https://indiemap.org/
[KevinMarks] joined the channel
# [KevinMarks] You will hit some inconsistencies with "fetch the url if there's no content only summary" from sites like adactio.com or daringfireball.net where they have some posts that are effectively short comments on external links, but I think the ux still works of expanding name -> summary -> content. What you may find is that the linked-to content doesn't have mf2 markup
# [KevinMarks] The difference may be if you iframe the page rather than parsing it?
[LewisCowles] joined the channel
# [LewisCowles] [Zegnat] it won't, I tried a few flags before settling on the one I PR'd ;)
# Zegnat If you have a moment / have the PHP env still available, I would love to see your output for https://gist.github.com/Zegnat/a94489e9b7d5501193e724e336bc6052#file-lookingforoptions-php
# [LewisCowles] The no warning flag might work. It's not one of the ones I tested.
# [LewisCowles] Scrolling way up there was another libxml flag I tried which did not work
# [LewisCowles] I wonder if we can find a branch in the PHP extension that uses your flag or mine
# [LewisCowles] It's a fine flag if it passes. I think I may have mis-interpreted what you were saying as "any `DOMDocument->load($string, $flag)` works. You mean most LIB_XML flags apart from `LIBXML_NOBLANKS` right?
[chrisaldrich] joined the channel
# [LewisCowles] Which ones are using < 2.7.0? I only tested it locally in my docker + travis
# [LewisCowles] I'm fine with not using that flag
# [LewisCowles] Zegnat++
# [LewisCowles] It's solid compassionate thinking
# [LewisCowles] oh that could be me too. I keep a dreamhost around and it's what I use GWG work on
# [LewisCowles] The PR looked fine. I've not contributed so, I don't think I can give any meaningful review. Your work seemed good.
# [LewisCowles] Peter / Chun-Sheng has another open PR. I think it should wait until after your one merges, but they are a nice person that has contributed to some other PHP I've done in the past
# [LewisCowles] their GitHub is Peter279k, their actual name is not Peter. I'll never cease to be amazed at pseudonyms that are actually names
# [LewisCowles] I have a test WP which has a few WordPress plugins for IndieWeb. You're my reference point on how some things should work
# [LewisCowles] Weather plugin tests were all on a DreamHost
# [LewisCowles] I think you use php-mf2 don't you GWG?
# [LewisCowles] Oh he's always doing those sorts of PR's, but occasionally I've ignored them and then needed to manually do the work later
# [LewisCowles] Blindly using latest serves little benefit, but how far behind are we. Some of the smaller changes in visibility and function signatures seemed solid
# [LewisCowles] It seems like an unfortunate smell
# [LewisCowles] i.e. what you want is a wide range of PHP with test coverage for each
# [LewisCowles] but the way of getting it is hacking phpunit version
# [LewisCowles] It's documented, so it's not wrong
# [LewisCowles] but is it not possible to use composer.json and let travis pick the right version?
# [LewisCowles] that would then defer the lock file creation to the tests
# [LewisCowles] I'd never seen that notation
# [LewisCowles] TIL: Zegnat++
# [LewisCowles] I'm not sure I'll remember it. Will it cope with phpunit changing signatures?
# [LewisCowles] even then the "hacks" can be stripped into checks in a base test class, so you extend that instead of PHPUnit, and handle changes there
# [LewisCowles] Nothing is impossible, but the effort...
# Zegnat peter279k has fixed this before though, it is all pretty straight forward, just something you need to take some time out for to test: https://github.com/paragonie/random_compat/pull/163
# [LewisCowles] See, he's a nice guy
# [snarfed] vilhalmer: eg re "explore how common different mf2 objects are in the wild" specifically, check out https://indiemap.org/docs.html#mf2_by_sites
leg and [jgmac1106] joined the channel
# Zegnat Interesting extension for checking on site performance: https://addyosmani.com/blog/web-vitals-extension/
nickodd, [John_Payne], [prtksxna], [schmarty], flex14, geoffo, [snarfed], KartikPrabhu, [KevinMarks], oenone and gRegorLove joined the channel; nickodd left the channel
# @sonicrocketman This Site Supports Webmentions! https://ift.tt/3djJcyu (twitter.com/_/status/1266847705583833093)
[chrisaldrich], [John_Payne] and xsteadfastx joined the channel