leanprover / leanprover/fp-lean
Combining IO and Reader uses String.separate instead of String.intercalate
Nobody has claimed this yet.
- Dominant language
- Lean
- Stars
- 192
- Forks
- 73
- PR merge metrics
- No merged PRs in 30d
Description
Please quote the text that is incorrect:
IO.eprintln s!"Didn't understand argument(s) {" ".separate args}\n"
In what way is this incorrect?
I don't know if this counts as a technical error or typo, as it pertains to non-working code.
Unless I'm mistaken, String.separate does not exist, and the correct code would be String.intercalate. Only the latter works for me and I cannot find String.separate in the lean4 source code.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the quoted example containing String.separate in the FP in Lean materials. Confirm that String.intercalate is the working name, then update the incorrect reference and verify that the example no longer uses the nonexistent function.
Written by the indexing model from the issue text.
Assessment
- Domain
- documentation
- Issue type
- Documentation
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100