Skip to content

Update constants docs#3058

Open
BNAndras wants to merge 4 commits intoexercism:mainfrom
BNAndras:update-constants
Open

Update constants docs#3058
BNAndras wants to merge 4 commits intoexercism:mainfrom
BNAndras:update-constants

Conversation

@BNAndras
Copy link
Copy Markdown
Member

@BNAndras BNAndras commented Apr 5, 2026

Related to #3042


## What can be a constant

Constants can only hold booleans, numbers, strings, and runes.
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.

Suggested change
Constants can only hold booleans, numbers, strings, and runes.
Constants can only hold Booleans, numbers, strings, and runes.

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.

I’d agree since it’s named after a person, but Go uses booleans. We also use it lowercased in our docs (see https://exercism.org/tracks/go/concepts/booleans)

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.

I think Bethany set me straight on this. Go (and Python) has a bool data type that represents a Boolean value, similar to an int type that represents an integer. There is no "Integer" and there is no "boolean".

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.

Yep. I’m fine with the change. My thought was that this should also be updated across the track where relevant. That can happen before or after this is merged.

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.

Definitely. Doing it as a separate PR might be cleaner.

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.

There's a few differences between the intro and about. I'm not sure if those are intentional. If the two are fairly close, it might be worth making them the same.

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.

I’d rather have different content for the two files, but the overlap is fairly significant here. We might as well make them the same.

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.

I'm all for making them different ... where there's a value to that difference, eg omitting some sections from the intro to make it simpler. But if the content is largely the same, there's no reason not to make them entirely the same.

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.

Agreed. I’ll update the docs Monday.

BNAndras and others added 3 commits April 4, 2026 21:24
Co-authored-by: Isaac Good <IsaacG@users.noreply.github.com>
Co-authored-by: Isaac Good <IsaacG@users.noreply.github.com>
Co-authored-by: Isaac Good <IsaacG@users.noreply.github.com>
@IsaacG IsaacG added x:action/improve Improve existing functionality/content x:module/concept Work on Concepts x:type/docs Work on Documentation x:size/medium Medium amount of work x:rep/medium Medium amount of reputation labels Apr 5, 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/docs Work on Documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants