Closed trevorld closed 4 years ago
Would love to see this merged :)
@MinchinWeb: If you want, you could submit a PR to address the issue upstream in Pelican itself, as I noted in a comment last month.
@justinmayer : Pull Request (pelican#2609) has been created! Also, I cleaned up a few failing-on-Windows tests (pelican#2609). As well, I have two languishing pull requests: pelican#2431 that standardizes how paths in settings are built, and pelican#2428 that adds Windows testing to Travis-CI; can you let me know what's needed to move these last two along? Thanks!
@Scheirle : Even with these pull requests submitted to Pelican, I think it would still be worthwhile to merge this one too.
Thanks to @MinchinWeb, the fix for this issue has been merged upstream and has been automatically published as Pelican 4.1.1, courtesy of AutoPub. ✨
Or you could wait to see if 'pelican' fixes their default status code. Without a fix here or there this plugin won't work.
https://github.com/getpelican/pelican/issues/2558