Closed scabug closed 12 years ago
Imported From: https://issues.scala-lang.org/browse/SI-5677?orig=1 Reporter: Jed Wesley-Smith (jedws) Affected Versions: 2.9.2
@som-snytt said: I came across: https://github.com/scala/scala/commit/f578ff88d2514bf94c46cfb8c0e33236c2b2fcf1
As a brief diversion from real work, implemented Damerau-Levenshtein
and ran it on trunk to elicit obvious misspellings. Unfortunately
they're mostly in places like compiler comments which real people never
see, but I fixed them anyway. All those English Lit majors who peruse
our sources are sure to be pleased.
As a Lit major, I really was pleased, but it would be even more cool to have a tool that generated crossword puzzles from scaladoc, so that we could learn stuff while having a bit of extraneous fun. (Aside from the intrinsic fun of learning Scala.)
Jed Wesley-Smith (jedws) said: While that commit is deeply satisfying to not only Lit majors, it doesn't appear to have fixed this particular error.
Indeed as at the time of writing the error is still there
@som-snytt said: What I should have said: Maybe it would be useful to scan for typos periodically in scaladoc comments. If there were a build target to do it, applying the corrections could be a community effort.
@heathermiller said: Hi guys. Thanks for spotting this. It's fixed here: https://github.com/scala/scala/pull/415. Also of note- since the github move, we all (committers included) now submit pull requests to scala/scala. So, if you'd ever like to fix a typo or expand API documentation in any way, please feel free to submit a pull request, you'll be credited in the scala/scala github repo for the fix/commit :)
@heathermiller said: Fixed in pull request #415 / commit b5d583063e995a2054fa3fde3b190ee4ffa843f5
Jed Wesley-Smith (jedws) said: thanks Heather, most appreciated! I'll try and submit a pull next time.
only one m, cumulative
also can't edit priority, should be trivial