Skip to content

Update grammar and wording in the Iterable trait tutorial#3305

Open
neuroevolutus wants to merge 1 commit into
scala:mainfrom
neuroevolutus:iterable-trait-grammar-and-wording
Open

Update grammar and wording in the Iterable trait tutorial#3305
neuroevolutus wants to merge 1 commit into
scala:mainfrom
neuroevolutus:iterable-trait-grammar-and-wording

Conversation

@neuroevolutus

Copy link
Copy Markdown
Contributor

One small change in particular that may be worth double-checking was the replacement of a double hyphen in the Markdown source with an em dash. The change precedes the words "dropping the ordering..." in the bullet for "Element retrieval".

@neuroevolutus neuroevolutus marked this pull request as draft June 18, 2026 21:52
@SethTisue SethTisue self-assigned this Jun 18, 2026
@neuroevolutus neuroevolutus force-pushed the iterable-trait-grammar-and-wording branch from a279795 to 16792b1 Compare June 19, 2026 05:52
@neuroevolutus neuroevolutus force-pushed the iterable-trait-grammar-and-wording branch from 16792b1 to 2175f65 Compare June 19, 2026 21:45
@neuroevolutus

Copy link
Copy Markdown
Contributor Author

I think the HTMLProofer step of the CI is failing (at least in part) due to a redirect from an old LAMP EPFL GitHub link. Edit: Never mind; the proofer runs seem to be flaky since that site seemed to yield no error on a subsequent run.

Also, the top-level page for the scala-exercises.org site seems to lead to a non-English page. It looks dubious, but, then again, I may be wrong since I don't know the language in which it's written.

The Gluon site is fine although the https://gluonhq.com/products/mobile/attach/ page in particular no longer exists.

I'm a bit befuddled since I haven't changed any of the links.

@SethTisue

Copy link
Copy Markdown
Member

Yeah you can just ignore the htmlproofer failures, I'll deal with it.

@neuroevolutus neuroevolutus marked this pull request as ready for review June 20, 2026 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants