Skip to content

Sync: exercise instructions #299

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 9 commits into from
Apr 28, 2025
Merged

Sync: exercise instructions #299

merged 9 commits into from
Apr 28, 2025

Conversation

rabestro
Copy link
Member

@rabestro rabestro commented Apr 27, 2025

./bin/configlet sync --update --docs 

Description of Changes

This pull request focuses on improving clarity, grammar, and consistency across various exercise instruction texts:

  • Pascal's Triangle: Corrected a grammar issue in the introduction text ("lookup" to "look up").
  • Egg Count Exercise: Updated a decimal value from 16 to 8 for better context alignment.
  • Sieve Exercise: Simplified instructions, reorganized content, and added examples for better comprehensibility.
  • Phone Number Cleanup: Removed redundant wording for improved readability.
  • Meetup Instructions: Updated count of week values from five to six for accuracy.
  • Luhn Exercise: Reorganized and enhanced explanations for clarity with better examples.
  • Grains Exercise: Refactored instructions for more concise and structured content.
  • Anagram Exercise: Fixed grammatical inconsistencies and ensured concise phrasing.
  • Alphabet Terminology: Replaced "Roman alphabet" with "Latin alphabet" to align with accurate terminology.

Checklist

  • Tests are passing.
  • Relevant documentation has been updated.
  • New or updated documentation has been reviewed.

Jegors Cemisovs added 9 commits April 27, 2025 23:16
Replaced "Roman alphabet" with "Latin alphabet" in the instructions for consistency and accuracy. This ensures clarity and aligns with standard terminology.
Refactored the instructions for clarity and conciseness. Rephrased sentences to improve readability and fixed minor grammatical inconsistencies. The changes do not alter the exercise's functionality or purpose.
Simplified and streamlined the problem description, focusing on key details. Removed the narrative and reorganized text for better readability and comprehension.
Improved the phrasing of Luhn formula explanations for better readability. Added a clearer breakdown of valid and invalid number examples, reorganized sections, and ensured consistent formatting.
Corrected the description to list six week values instead of five. This ensures the instructions accurately represent all possible values for the week parameter.
Removed redundant wording in the description for clarity. This improves readability while maintaining the original intent of the instructions.
Refined the wording and structure of the Sieve of Eratosthenes instructions for clarity and added illustrative examples. This update improves comprehensibility and aligns explanations with the testing notes for better guidance.
Updated the displayed decimal number from 16 to 8 to ensure it matches the context. This change improves clarity and maintains alignment with the actual exercise details.
Corrected "lookup" to "look up" to improve grammar and clarity in the introduction documentation for the Pascal's triangle exercise. Ensures the text is more polished and professional.
@rabestro rabestro requested review from glennj and IsaacG April 27, 2025 20:33
@rabestro rabestro self-assigned this Apr 27, 2025
@rabestro rabestro added x:action/sync Sync content with its latest version x:type/docs Work on Documentation labels Apr 27, 2025
@glennj
Copy link
Contributor

glennj commented Apr 28, 2025

Did you do this with configlet sync?

@rabestro
Copy link
Member Author

./bin/configlet sync --update --docs 

The pull request description includes the command. Do we have any rules for syncing and other tasks?

@rabestro
Copy link
Member Author

Did you do this with configlet sync?

Yes, I did. IntelliJ IDEA automatically generates the PR description.

@glennj
Copy link
Contributor

glennj commented Apr 28, 2025

Do we have any rules for syncing and other tasks?

No. I was just surprised to see so many commits.

@glennj glennj added the x:rep/small Small amount of reputation label Apr 28, 2025
@glennj glennj merged commit 74a2c32 into main Apr 28, 2025
3 checks passed
@glennj glennj deleted the issue/sync branch April 28, 2025 11:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
x:action/sync Sync content with its latest version x:rep/small Small amount of reputation x:type/docs Work on Documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants