Skip to content

[documentation]: document the splitwhile tactic#881

Merged
strub merged 1 commit intomainfrom
doc-splitwhile-tactic
Feb 4, 2026
Merged

[documentation]: document the splitwhile tactic#881
strub merged 1 commit intomainfrom
doc-splitwhile-tactic

Conversation

@strub
Copy link
Member

@strub strub commented Feb 3, 2026

No description provided.

@strub strub self-assigned this Feb 3, 2026
@strub strub force-pushed the doc-splitwhile-tactic branch 2 times, most recently from 077986e to b38758c Compare February 3, 2026 08:17
Copy link
Member

@fdupress fdupress left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know we're still quite fragmented, but I think @Cameron-Low had pretty good documentation for intro-patterns and (more importantly in this context) code positions over in the ec-tactic repo.

Is it worth linking to that when talking about code positions? (Not ideal for the user.)
Or wait until we carry it over and add all links at that point? (That sounds like an excellent reason to never document code positions, which is bad.)

@strub
Copy link
Member Author

strub commented Feb 4, 2026

I know we're still quite fragmented, but I think @Cameron-Low had pretty good documentation for intro-patterns and (more importantly in this context) code positions over in the ec-tactic repo.

Is it worth linking to that when talking about code positions? (Not ideal for the user.) Or wait until we carry it over and add all links at that point? (That sounds like an excellent reason to never document code positions, which is bad.)

Yes, this is a question I asked to myself. We clearly want to link back to the relevant chapters/sections. Maybe should we draft a TOC so that we can start linking to them. Better to do it on the fly than to do it at the end.

@strub strub force-pushed the doc-splitwhile-tactic branch from 0bc8ea1 to 077986e Compare February 4, 2026 14:29
@strub strub force-pushed the doc-splitwhile-tactic branch from 077986e to 61063dd Compare February 4, 2026 14:50
@strub strub enabled auto-merge (rebase) February 4, 2026 14:50
@strub strub merged commit c148172 into main Feb 4, 2026
17 checks passed
@strub strub deleted the doc-splitwhile-tactic branch February 4, 2026 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants