| Geth | doc: tbrowder++ created pull request #4750: Add description of new Rakudo test 'exits-ok' |
00:16 | |
| tbrowder | arg, i think i messed it up again... | 00:23 | |
| [Coke] | You should be able to just keep pushing on the same branch and the PR will automatically update. | 00:32 | |
| Geth | doc: coke++ created pull request #4751: Start a PR |
00:41 | |
| [Coke] | That is just a sample PR. I have a fork. I did a new branch, did one commit, opened a PR. | 00:42 | |
| did another commit. Pushed it. it *immediately* shows up on the PR that was already opened. | |||
| and there was no message on the push about opening a new PR. | 00:43 | ||
| if you look at the PR, you can see where I marked the PR as draft - that was after the initial open of the PR. the commit right after that - that was just from me pushing to git@github.com:coke/doc.git on the coke/some-pr branch. | 00:44 | ||
| When I was getting my old fork clean, I did have to switch the main branch from master to main to track the repo. (it's been a while since I touched it) | 01:26 | ||
| tbrowder | hm, the PR looks correct now and ready to merge. | 03:08 | |
| [Coke] | still has trailing whitespace | 04:31 | |
| line 231 | 04:32 | ||
|
09:59
wayland76 joined,
wayland left
|
|||
| tbrowder | i don't see it on my host, i'll try to edit on github | 11:51 | |
| [Coke] | with the updated one, I can see it on "test L<C<exits-ok>|/type/Test#sub_exits-ok>. " at least. | 14:30 | |
| what editor do you use for local edits? in vi, you can use slash space $ to find any spaces at the end of a line - not all whitespace, but the typical one. | 14:31 | ||
| tbrowder | vi, i'll do that... | 14:43 | |
| nada! | 14:49 | ||
| is it possible it's a non-breaking space? unicode has several kinds of "spaces" | 14:55 | ||
| i'll try another edit on github in a while... | 14:57 | ||
| it worked this time. it's weird i couldn't see the extra space locally | 15:29 | ||
| it's still failing on whitespace, but i'm not sure it's due to my code. i give up for now. | 16:40 | ||
|
18:22
arkiuat joined
|
|||
| Geth | doc/arkiuat-patch-1: c20b3d41e9 | (Eric Forste)++ (committed using GitHub Web editor) | doc/Language/control.rakudoc control.rakudoc: bad link to #Closures section a link plainly intended to direct to the #Closures section instead pointed to the #Blocks_and_lambdas section. Corrected. |
18:35 | |
| doc: arkiuat++ created pull request #4752: control.rakudoc: bad link to #Closures section |
18:36 | ||
| ¦ doc: arkiuat self-assigned The whole explanation for the difference between block and closure is LTA github.com/Raku/doc/issues/3670 | 18:38 | ||
|
18:39
finanalyst joined
19:09
lizmat joined
19:10
Geth joined
19:13
arkiuat left
19:20
arkiuat joined
19:24
arkiuat left
19:41
arkiuat joined
19:46
arkiuat left
19:58
arkiuat joined
20:06
arkiuat left
|
|||
| [Coke] | tbrowder: can you run the trailing whitespace test locally? | 20:26 | |
|
20:26
arkiuat joined
20:31
arkiuat left
|
|||
| tbrowder | yes. just a moment | 20:34 | |
|
20:44
arkiuat joined
|
|||
| tbrowder | what test should i use? there is no modified branch so the *modified.sh doesn’t work | 20:46 | |
| make | 20:47 | ||
| disregard “make”… | 20:48 | ||
| i | 20:50 | ||
| i’m running “make test”… | |||
| [Coke] | RAKULIB=. TEST_FILES="<name of file>" t/04-trailing-whitespace.rakutest | 20:55 | |
| much faster | |||
| tbrowder | ah, i forgot... | 21:21 | |
| git br | 21:29 | ||
| [Coke] my local system's repo main and tb-exists-ok are not in proper synch with docs/main. i don't know how to fix it quickly. what i will do is make sure and save my code and then completely delete my local and remote doc tree and start over unless you have a better solution. | 21:36 | ||
|
21:54
finanalyst left
|
|||
| [Coke] | it's line 234 of grep -n '\s$' doc/Language/testing.rakudoc | 22:40 | |
| "not in sync" ? | |||
| the extra whitespace is just before the preamble line you added. | 22:41 | ||
| and it's just a regular space | |||
| You only edited one file, even if you did your branch off an older commit, it still applies cleanly. | 22:42 | ||
| (which the PR tells you at the end after the failed test) | |||