Documentation Channel for #raku | This channel is logged | Roadmap: github.com/raku/doc/wiki
Set by [Coke] on 23 May 2022.
03:10 rf left 12:03 NemokoschKiwi joined 12:13 NemokoschKiwi left
[Coke] 32 tickets left for end of March. 14 are checklists, 4 are Kaiepi's PRs that hopefully require very little work. 13:52
m: say "{(32 / (Date(2023,3,31)-Date.today))} tickets/day"; 13:54
2. it's 2 (and there's a typo in that line.)
and I just closed 2 that apparently lizmat had already completed. :) 14:00
Geth ¦ doc: coke self-assigned encoding list missing in some locations github.com/Raku/doc/issues/4090
[Coke] cfa: any outstanding issues on the links PR? 14:09
I addressed at least some of the concerns you had. 14:10
and regarding the testing of doc-website templates - I think the way I did it makes more sense because you have dependencies to run the tests that you would have already had to satisfy on raku/doc - if you instead pull the tests into doc-website via the clone of raku/doc, you may then not be able to run them unless you install the deps (and the tests may require work to deal with being run out of a 14:13
different relative directory)
Geth doc/main: 9b912d27c6 | Coke++ | README.md
clarify
15:15
doc/main: 838edc4e77 | Coke++ | README.md
Note frequency of site updates isn't immediate.
15:19
doc/main: a8e54a3dfa | cfa++ | doc/Language/grammar_tutorial.rakudoc
Fix typo
16:05
16:06 cfa joined
cfa morning 16:07
[Coke] la 16:48
16:48 rf joined
cfa [Coke]: links pr looks good from a preliminary run this morning 17:01
lots of failures, though it's possible to address these with some automated rewrites
as for the content tests, i suppose that makes sense. with this in place, though, we'd have each repo cloning the other, which seems messy and coupled. 17:02
i understand that you don't want to duplicate the tests themselves though 17:03
back to the links test: i wonder whether the test itself should skip over 'raku', 'perl' and other tokens that are likely to be false positives. it might also be worthwhile to add a negative look ahead for 'method' so that 'Str method' isn't treated as if it should link to /type/Str 17:07
[Coke] all good points. 17:08
cfa i guess i'm not sure whether it makes sense to merge that links test before addressing most of the failures 17:10
or whether to maintain it in draft, address most of them, then commit 17:11
btw, you've got a rewrite in the pr -- not sure whether that's accidental
[Coke] a rewrite? 17:12
If you mean force push, yes, probably 17:13
cfa no, a tweak to the documentation to satisfy the test
github.com/Raku/doc/pull/4256/file...081703aff3
[Coke] Figured the end result of this branch is one commit to add the test, one commit to make it pass. 17:16
cfa one commit to make it pass might make sense, though it'll touch a lot of files 17:17
[Coke] we could back that out and just have the test, I know you had another branch with fixes.
cfa here: Failed 118/416 subtests
[Coke] well, N commits.
cfa (but some of those rewrites are likely incorrect)
188 files changed, 2678 insertions(+), 2678 deletions(-) 17:18
some of those failures are C<Raku> etc. 17:20
and some are rewrites that i missed with these simple patterns
cfa reruns the test in the type-links branch
Geth doc/type-links: b7ab3cd77d | (Will Coleda)++ | doc/Type/Bool.rakudoc
Revert "clean up test descriptions"

This reverts commit a8963e1d620075b044ffcfd482adda5827af3a05.
only for doc/Type/Bool.rakudoc
17:23
[Coke] there, now we're back to just the test 17:24
cfa Failed 237/416 subtests 17:26
oh, that's before you did the above
so those rewrites address approx. 119 failures
as for the #index-entry stuff: if the tooling can re-create those older targets, great; if not, i'd suggest we merge this or something similar in the near future. as it stands, a lot of cross-references are broken, which leads to a frustrating user experience 17:34
(and if the targets can be regenerated but not in the near future, it might be worth merging something like this then rolling back once generation is fixed) 17:35
i defer to you on that 17:36
Geth doc/type-links: e52b6c87e5 | (Will Coleda)++ | xt/rakudoc-types.rakutest
Don't complain about perl or raku
17:39
[Coke] let's get it working in place, keeping old links working if possible 17:41
ah, see your q to fin. let's see ehat he says 17:42
perl/raku shaves a few failures, not many
cfa agreed re: old links 17:43
[Coke] if the link is something like C<Str> method - This should prob be L<Str method|...>
and so we should complain, but the fix may be not the simple one the test recommends. 17:44
cfa i suppose so 17:46
oh, missed some rewrites 17:48
114 failures here but i'll rerun the test in a few
[Coke] We should add the main suggestion to the styleguide, also, and any others like for methods. 17:54
Checking the text after the link will be harder.
since we're walking the pod structure, and would have to save state... Guessing it's not worth figuring out the coding if we're already down to 114 total. 17:55
would be nifty, though.
cfa hopefully <114 17:56
but i'm not sure how many of these are incorrect rewrites
the changes that i've spotchecked have been correct
okay, Failed 65/416 subtests 18:00
[Coke] that's 65 files, not 65 instances, yes? 18:01
cfa # You failed 65 tests of 416
this is the output of prove
so in doc/Type/independent-routines.rakudoc, # You failed 6 tests of 70 18:02
looks like there're very few files that are failing
oh, no, you're right: 65 files 18:05
[Coke] yah, doing each file as a collection of subtests is much easier to write, but harder to count. 18:15
1/6 is pretty good considering we started with failing so much 18:16
cfa [Coke]: take a look at github.com/cfa/raku-doc/tree/norma...type-links 18:23
i haven't tried rendering this yet
but you can see the rewrites there
and test them locally
(239 changed files with 2,772 additions and 2,772 deletions) 18:24
this is a child of your type-links branch btw 18:25
[Coke] nice. 18:29
19:06 NemokoschKiwi joined 19:19 NemokoschKiwi left 19:33 cfa left 22:01 raiph joined 23:34 raiph left