Skip to content

Update strings-package concept docs#3061

Open
BNAndras wants to merge 3 commits intoexercism:mainfrom
BNAndras:update-strings-package-concept
Open

Update strings-package concept docs#3061
BNAndras wants to merge 3 commits intoexercism:mainfrom
BNAndras:update-strings-package-concept

Conversation

@BNAndras
Copy link
Copy Markdown
Member

@BNAndras BNAndras commented Apr 7, 2026

Related to #3042

| Modifying | [ReplaceAll](https://pkg.go.dev/strings#ReplaceAll) | Replace all occurrences of a substring |
| Modifying | [TrimSpace](https://pkg.go.dev/strings#TrimSpace) | Remove leading and trailing whitespace |
| Splitting or Joining | [Join](https://pkg.go.dev/strings#Join) | Join a slice of strings with a separator |
| Splitting or Joining | [Split](https://pkg.go.dev/strings#Split) | Split a string into parts on a separator |
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

FieldsSeq and SplitSeq might be work mentioning.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

That's fine by me, but the additional code examples might need some explanation given that range iteration is covered way later in the syllabus. Slices are also later in the syllabus but they're not as far so they're probably okay.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

sigh Building a syllabus where you can only mention prior topics is hard.

I think a comment mentioning "Don't worry about this syntax too much; we discuss it in the range concept later" should suffice.

Comment thread concepts/strings-package/about.md
Comment thread concepts/strings-package/about.md Outdated
BNAndras and others added 2 commits April 6, 2026 22:21
Co-authored-by: Isaac Good <IsaacG@users.noreply.github.com>
@@ -1,9 +1,41 @@
# Introduction
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should the files be synced?

@IsaacG IsaacG added x:action/improve Improve existing functionality/content x:module/concept Work on Concepts x:type/content Work on content (e.g. exercises, concepts) x:type/docs Work on Documentation x:size/medium Medium amount of work x:rep/medium Medium amount of reputation labels Apr 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

x:action/improve Improve existing functionality/content x:module/concept Work on Concepts x:rep/medium Medium amount of reputation x:size/medium Medium amount of work x:type/content Work on content (e.g. exercises, concepts) x:type/docs Work on Documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants