Skip to content

Updated explanation for Keyword Matching (3.3.2) and examples for Free Named Arguments (2.5.1.2)#80

Open
bennyvw wants to merge 5 commits intomainfrom
updated-explanation-and-examples
Open

Updated explanation for Keyword Matching (3.3.2) and examples for Free Named Arguments (2.5.1.2)#80
bennyvw wants to merge 5 commits intomainfrom
updated-explanation-and-examples

Conversation

@bennyvw
Copy link
Contributor

@bennyvw bennyvw commented Mar 2, 2026

Important note: the image of the keyword documentation of the Set To Dictionary keyword has to be changed. I did not know how to create the 'splitted' image with the waves in between, so please help me out on that.

Checklist

  • Label set: impact:major or impact:minor
  • PR title contains: Subject + Chapter scope + Topic
  • One bullet per issue/change (see “Issues”)
  • Each bullet includes a chapter (Ch. X or Ch. X.Y)
  • Uses imperative verbs (Fix / Remove / Rephrase / Clarify)
  • Descriptions are short (3–10 words)

Major change checklist

  • Exam Committee reviewed and confirmed exam impact (link/note):
  • Training Providers informed (date + link):
  • Effective date (countdown ends): YYYY-MM-DD

Issues

Herewith issues and are solved.

Copy link

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

Updates the syllabus documentation to reflect Robot Framework’s updated keyword matching behavior (RF 7.1+) and replaces outdated Run Process examples (affected by Process library changes) with Set To Dictionary examples.

Changes:

  • Replaced the chapter 2 keyword interface example keyword from Run Process to Set To Dictionary, including updated examples for varargs and free-named-args usage.
  • Updated the chapter 3 keyword matching algorithm description to reflect the new BDD-prefix handling behavior.
  • Added a new screenshot asset for Set To Dictionary keyword documentation.

Reviewed changes

Copilot reviewed 2 out of 4 changed files in this pull request and generated 2 comments.

File Description
website/docs/chapter-02/05_keyword_interface.md Switches the example keyword to Set To Dictionary, updates varargs/free-named-args explanations and examples, and updates the referenced screenshot.
website/docs/chapter-03/03_user_keyword.md Updates the described keyword matching algorithm order (BDD prefixes vs full match).
website/static/img/Set_To_Dictionary_part_1.png Adds the new screenshot used by chapter 2.

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


This keyword has one :term[Mandatory Arguments]{tooltipMd="An **Argument** that must be set.<br/>See [Mandatory Args](chapter-03/03_user_keyword.md)"} `command` which can be called positionally or by name.
The latter two arguments are optional.
This keyword has one :term[Mandatory Arguments]{tooltipMd="An **Argument** that must be set.<br/>See [Mandatory Args](chapter-03/user_keyword#3351-defining-mandatory-arguments)"} `dictionary` which can be called positionally or by name.
Copy link

Copilot AI Mar 2, 2026

Choose a reason for hiding this comment

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

The tooltip link target chapter-03/user_keyword#3351-defining-mandatory-arguments doesn’t match the internal linking convention used elsewhere in the docs (e.g., chapter-03/03_user_keyword.md#...) and is likely a broken link. Update it to point to the actual user keyword chapter file/anchor used in the rest of the syllabus.

Suggested change
This keyword has one :term[Mandatory Arguments]{tooltipMd="An **Argument** that must be set.<br/>See [Mandatory Args](chapter-03/user_keyword#3351-defining-mandatory-arguments)"} `dictionary` which can be called positionally or by name.
This keyword has one :term[Mandatory Arguments]{tooltipMd="An **Argument** that must be set.<br/>See [Mandatory Args](chapter-03/03_user_keyword.md#3351-defining-mandatory-arguments)"} `dictionary` which can be called positionally or by name.

Copilot uses AI. Check for mistakes.

This keyword has one :term[Mandatory Arguments]{tooltipMd="An **Argument** that must be set.<br/>See [Mandatory Args](chapter-03/03_user_keyword.md)"} `command` which can be called positionally or by name.
The latter two arguments are optional.
This keyword has one :term[Mandatory Arguments]{tooltipMd="An **Argument** that must be set.<br/>See [Mandatory Args](chapter-03/user_keyword)"} `dictionary` which can be called positionally or by name.

This comment was marked as resolved.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The link chapter-03/03_user_keyword.md led to a 404..

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@GerwinLaagland
Copy link
Collaborator

So had a quick look as to why this seems to fail. The code checks the headers that are expected to be updated for the readme file:

This repo has a few files that are generated automatically from the documentation (most visibly the Table of Contents in README.md).

When headings in the docs change, a script rebuilds the numbering and the TOC so everything stays consistent. The GitHub workflow runs that script on every PR. If the script detects that it would update any generated file, it fails the check to signal:

“Some generated files weren’t regenerated and committed yet.”

In this PR, the script shows a diff for README.md, meaning the README TOC is currently out of sync with the docs.


How to fix

Please regenerate the generated files locally and commit the results:

python tools/gen_numbering.py

Then commit whatever files changed (often README.md, and sometimes a few others updated by the script) and push the commit to this PR.

Once the regenerated output is committed, the CI check should pass. (Thanks ChatGPT for helping me out. 😅)

@github-actions
Copy link

github-actions bot commented Mar 2, 2026

@GerwinLaagland GerwinLaagland added the impact: major Substantive; triggers the major-change workflow label Mar 4, 2026
@github-actions
Copy link

github-actions bot commented Mar 4, 2026

@github-actions
Copy link

github-actions bot commented Mar 4, 2026

@github-actions
Copy link

github-actions bot commented Mar 4, 2026

@GerwinLaagland GerwinLaagland changed the title Updated explanation and examples Updated explanation for Keyword Matching (3.3.2) and examples for Free Named Arguments (2.5.1.2) Mar 4, 2026
@github-actions
Copy link

github-actions bot commented Mar 4, 2026

@github-actions
Copy link

github-actions bot commented Mar 4, 2026

1 similar comment
@github-actions
Copy link

github-actions bot commented Mar 4, 2026

@github-actions
Copy link

github-actions bot commented Mar 4, 2026

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

impact: major Substantive; triggers the major-change workflow

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants