Skip to content

Fix glossary and typos#90

Merged
Snooz82 merged 6 commits intomainfrom
fix-glossary-and-typos
Apr 24, 2026
Merged

Fix glossary and typos#90
Snooz82 merged 6 commits intomainfrom
fix-glossary-and-typos

Conversation

@Snooz82
Copy link
Copy Markdown
Member

@Snooz82 Snooz82 commented Apr 23, 2026

fixed the review comments from Pekka and fixed multiple typos or grammar issues.

This PR polishes the RFCP website content by correcting typos/grammar and refining glossary terminology so the syllabus pages and glossary tooltips read more consistently across the site.

Changes:

  • Updated glossary.json definitions and added/adjusted several glossary entries (e.g., dictionary/list variable syntax, IF statement, suite directory).
  • Corrected typos and grammar across multiple syllabus chapters, overview pages, and the example exam page.
  • Aligned repeated learning objective phrasing across docs, README.md, and LOs.csv.

Snooz82 and others added 2 commits April 23, 2026 18:58
Co-authored-by: Copilot <copilot@github.com>
@github-actions
Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR polishes the RFCP website content by correcting typos/grammar and refining glossary terminology so the syllabus pages and glossary tooltips read more consistently across the site.

Changes:

  • Updated glossary.json definitions and added/adjusted several glossary entries (e.g., dictionary/list variable syntax, IF statement, suite directory).
  • Corrected typos and grammar across multiple syllabus chapters, overview pages, and the example exam page.
  • Aligned repeated learning objective phrasing across docs, README.md, and LOs.csv.

Reviewed changes

Copilot reviewed 25 out of 26 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
website/static/glossary/glossary.json Refines glossary wording; adds/updates entries and aliases to improve term clarity in the site glossary/term tooltips.
website/docs/overview.md Fixes punctuation in provider description.
website/docs/learning_objectives.md Fixes grammar in LO table entries for consistency.
website/docs/example-exam/Example-exam.mdx Fixes typos/grammar in the example exam intro sentence.
website/docs/chapter-05/02_control_structures.md Minor grammar correction in inline IF explanation.
website/docs/chapter-05/01_advanced_variables.md Fixes typo (“execution”).
website/docs/chapter-04/05_skip.md Grammar fix in skip example sentence.
website/docs/chapter-04/04_tags.md Text refinements around tag description and reserved tag warning.
website/docs/chapter-04/03_init_files.md LO grammar fix + wording tweak in settings list.
website/docs/chapter-04/02_teardowns.md Pluralization correction (“tests
website/docs/chapter-03/04_datadriven.md Grammar fixes in data-driven explanation.
website/docs/chapter-03/03_user_keyword.md Grammar/plural fixes and typo correction in a comment/TODO block.
website/docs/chapter-03/02_variables.md Fixes punctuation/parentheses and improves sentence readability.
website/docs/chapter-03/01_resource_file.md Grammar/capitalization fixes.
website/docs/chapter-02/06_writing_test.md Multiple grammar fixes and improved phrasing in examples/explanations.
website/docs/chapter-02/05_keyword_interface.md Grammar/wording fixes for clarity and correctness.
website/docs/chapter-02/04_keyword_imports.md Grammar fixes in library/import explanations.
website/docs/chapter-02/03_executing.md Grammar fixes and status formatting corrections.
website/docs/chapter-02/02_suitefile_syntax.md Link phrasing fix + capitalization/grammar corrections.
website/docs/chapter-02/01_suitefile.md Terminology and wording alignment (“Suite Directories”, “suite metadata”).
website/docs/chapter-01/04_styles.md Grammar fixes in specification style explanation.
website/docs/chapter-01/03_syntax.md Grammar/punctuation fixes in syntax description.
website/docs/chapter-01/02_architecture.md Minor grammar correction (“process the model”).
website/docs/chapter-01/01_purpose.md Minor pluralization fix (“failures”).
README.md Synchronizes LO wording with docs.
LOs.csv Synchronizes LO wording with docs/README.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread website/static/glossary/glossary.json Outdated
Comment thread website/docs/chapter-02/04_keyword_imports.md Outdated
Comment thread website/docs/chapter-03/04_datadriven.md Outdated
@github-actions
Copy link
Copy Markdown

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 23, 2026 17:34
@github-actions
Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 25 out of 26 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread website/static/glossary/glossary.json Outdated
Comment thread website/static/glossary/glossary.json Outdated
Comment thread website/static/glossary/glossary.json Outdated
Comment thread website/static/glossary/glossary.json Outdated
Signed-off-by: René <snooz@posteo.de>
@github-actions
Copy link
Copy Markdown

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 24, 2026 10:24
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review is ineligible. To be eligible to request a review, you need a paid Copilot license, or your organization must enable Copilot code review.

@github-actions
Copy link
Copy Markdown

Removed redundant information about library types and a TODO comment.
Copilot AI review requested due to automatic review settings April 24, 2026 10:52
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review is ineligible. To be eligible to request a review, you need a paid Copilot license, or your organization must enable Copilot code review.

@github-actions
Copy link
Copy Markdown

Copy link
Copy Markdown
Collaborator

@GerwinLaagland GerwinLaagland left a comment

Choose a reason for hiding this comment

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

All feedback has been resolved, and the results look good to me. Approved.

@Snooz82 Snooz82 merged commit c4cfd7c into main Apr 24, 2026
3 checks passed
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.

3 participants