Time Nick Message 12:44 MTDiscord Custom 404 should definitely be possible, can you create an issue? I can look into it 12:59 rubenwardy Just make a 404.html file 13:01 MTDiscord on neocities it's not_found.html 13:34 MTDiscord GH pages will pick up a 404.html in the source directory. https://docs.github.com/en/pages/getting-started-with-github-pages/creating-a-custom-404-page-for-your-github-pages-site 17:15 MTDiscord I'm back 🙂 created https://github.com/minetest/dev.luanti.org/issues/133 (Add Hugo as a project dependency) and https://github.com/minetest/dev.luanti.org/pull/132 (Fix readme link) 17:20 MTDiscord ik, dunno if that will fight with hugo or not. also dont want to completely reimplement colors, etc 17:21 MTDiscord commented on pr 18:46 MTDiscord https://dev.luanti.org/worlds/ mentions "Schem" files, but it's the only page that does--is this a typo? If so, what's the better name? 18:48 MTDiscord Schematic files, or schematics for short 18:48 MTDiscord cba to look atm, but schematics, mts, etc 18:50 MTDiscord just going to poke you mark about implementing https://github.com/minetest/dev.luanti.org/issues/72 since thats a you thing 18:58 MTDiscord what does cba mean? 18:59 MTDiscord Yeah I'll add the new a11y check eventually, focusing on simple spell check today 🙂 18:59 MTDiscord cant be assed 18:59 MTDiscord or cant be arsed 19:01 MTDiscord interesting, urban dictionary only list the latter 🤷 https://www.urbandictionary.com/define.php?term=cba 19:08 MTDiscord I'm assuming "coredev" is generally used as one word? I'll keep it for now, but curious if folks want to maybe change it to two words for consistency with other projects? I don't see any need for it to be special in the long term 19:33 MTDiscord Any reason "unittests" is occasionally written as one word in prose? 19:36 MTDiscord No good reason, if you ask me. But I'm not going to complain as long as we write it the same way consistently. xD 19:38 MTDiscord Our unit tests are organized into a directory called unittests in our file hierarchy. Maybe that influences our documentation? 19:51 MTDiscord Yeah I noticed that, but I think in prose when we talk about or mention "unit tests" it should be written in sentence case. Was just curious if there was some other project-specific reason I was missing 19:52 MTDiscord unit tests should definitively be written as two words in prose 20:02 MTDiscord Cool, will update the issue with the same Looks like CI is still panicking on a warning about native HTML, I thought we removed that, I'll check it out. In any case, https://github.com/minetest/dev.luanti.org/pull/136 is up as a draft: Fix ~800 more spelling issues 20:03 MTDiscord Ah, I see https://github.com/minetest/dev.luanti.org/issues/96 was closed as not planned. Can we re-open it? I think native HTML is just fine and personally I'd encourage it where valuable (like comments, for example). 20:04 MTDiscord no, because its a valid warning that you should fix 20:04 MTDiscord (generall) 20:05 MTDiscord The warning is "native HTML", which isn't a problem if you know what the HTML is and you intended for it to be there. The warning was added to Hugo because Hugo is often used to host untrusted content--we don't do that, so we shouldn't be worried about it. See the issue for details 20:06 MTDiscord otherwise stuff like message is going to slip through 20:06 MTDiscord Ah, maybe, but I think the build itself might be wonky, or at least the rendered page wouldn't look good. I think now that we've imported stuff, we'll be manually reviewing the actual look and feel of most changes, right? 20:07 MTDiscord if you disable that warning, your not going to even know 20:07 MTDiscord I'm sure there's a middle ground, but I'm OK with letting things slip through if it makes it easier for us to do our work in other ways 20:08 MTDiscord we'll know because we look at the page and make sure it contains the content we intended, including the string as text on the page. If we're not doing that, we're being a bit too sloppy! 20:09 MTDiscord the irony is you made the pr fixing the stuff https://github.com/minetest/dev.luanti.org/pull/78/files 20:10 MTDiscord humans are humans and going to make mistakes 20:12 MTDiscord Yes, we will make mistakes. I'm ok with a typo or two getting to the website. I'm frustrated by this tool that's stopping me from adding comments to HTML files, especially because those comments are spell-checking directives Humans make mistakes, but tools aren't perfect. We can use informed judgement to decide which tools to use 20:12 MTDiscord and now that the stuff is fixed, we don't need the warning 😉 20:12 MTDiscord till more gets added 20:12 MTDiscord Do you mean more imported content or more manually written content? 20:12 MTDiscord either or, doesnt matter 20:12 MTDiscord cant expect everyone to know 20:13 MTDiscord Don't you think it's reasonable for us to manually review the rendered page of any future changes? 20:13 MTDiscord "us" being PR authors, at least 20:13 MTDiscord so your going to trust john doe that made a pr to look at the output? 20:13 MTDiscord your going to checkout and run every pr? 20:13 MTDiscord If the PR is small, I trust a reviewer to ensure there isn't anything suspicious 20:14 MTDiscord If the PR is big, I trust a reviewer to checkout and run it, or ask for it to be broken into smaller pieces 20:14 MTDiscord your comments would be slightly less insanty if we had something that generated a preview site for prs that where easy to look at, but we dont 20:15 MTDiscord example of where it saved me https://github.com/minetest/dev.luanti.org/pull/123. course i may be an idiot 🙃 20:15 MTDiscord insane is a strong word 😦 I'm just saying that we have a tool that is currently slowing us down in practice, right now. You're saying it's saving us from hypothetical situations. I know I fixed previously-imported files, but those were exactly that: files imported in a rush in a situation where we knew we'd be coming back and cleaning them up. Now that things are clean, I think we're in a very good state that we can trust native 20:15 MTDiscord HTML. Maybe the compromise is to show the warnings, but not block PRs on them? Or I should just find a way to allow HTML comments and nothing else? 20:16 MTDiscord Thanks for providing a concrete example. But I think "sanity lost" is enough for us to agree "we won't be doing things this way in the long term", does that sounds reasonable? 20:16 MTDiscord > hypothetical situations this is false, as demonstrated by the above links 20:16 MTDiscord Yes, I was mid-sentence when you posted that. I am mostly referring to the future now. I don't think this warning is helping us any more 20:17 MTDiscord And note I enabled panic on ALL warnings, not just native HTML ones. I'm happy to keep all other warnings, I just want to remove this native HTML one (at least for comments) 20:17 MTDiscord How about, for now, I find a way to allow just native HTML comments? Because without that I am struggling to disable the spell-checker for certain parts of the file 20:18 MTDiscord that seems fine. better to say open a small hole than the front door 20:22 MTDiscord looks like cspell supports /* cSpell:disable */, which is also a golang comment syntax. not sure if that helps at all https://cspell.org/configuration/document-settings/ 20:23 MTDiscord yeah I'm looking into other comment forms now, see also https://discourse.gohugo.io/t/warning-when-mixing-html-and-markdown/52280 for other forms and overall native HTML warning background 🙂 20:45 MTDiscord The weirdest thing is that there is no succinct comment syntax. The best I'm finding is https://stackoverflow.com/questions/45224766/how-to-comment-out-content-in-hugo, and even that is: {{% comment %}} hello world {{% /comment %}} vs absolutely wild. Plus Hugo 0.137.1 isn't even on NixOS lol 20:59 MTDiscord maybe that many core devs are german and like to fuse words (me included) lol 20:59 MTDiscord this is almost certainly why you see so many fuse words there 21:01 MTDiscord Makes sense! nbd for now. In any case, we need to merge https://github.com/minetest/dev.luanti.org/pull/137 before the spell-check currently 21:01 MTDiscord (for IRC folks, that PR fixes comment syntax) 21:13 MTDiscord https://github.com/minetest/dev.luanti.org/pull/136 - fix ~800 spelling issues - also ready for review 🙂