Skip to content

leap: Improve case descriptions#2301

Closed
m-dango wants to merge 1 commit intomainfrom
m-dango-patch-3
Closed

leap: Improve case descriptions#2301
m-dango wants to merge 1 commit intomainfrom
m-dango-patch-3

Conversation

@m-dango
Copy link
Copy Markdown
Member

@m-dango m-dango commented Jul 13, 2023

The descriptions had some typos. Those have been fixed along with some changes in wording for consistency and clarity.

The descriptions had some typos. Those have been fixed along with some changes in wording for consistency and clarity.
@IsaacG
Copy link
Copy Markdown
Member

IsaacG commented Jul 13, 2023

Those descriptions aren't meant to be complete sentences nor are they used as instructions.

Consider opening a discussion on the forum prior to sending PRs. In general, we like to chat about changes prior to receiving PRs.

@m-dango
Copy link
Copy Markdown
Member Author

m-dango commented Jul 13, 2023

Created forum topic, will make adjustments accordingly: https://forum.exercism.org/t/leap-changes-to-test-case-descriptions/6473

At the very least I don't believe "in" is the correct word to be used in these descriptions, and this was changed on one case in a previous PR: #1726

@IsaacG IsaacG closed this Sep 7, 2024
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