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 9 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 9 commits intomainfrom
updated-explanation-and-examples

Conversation

@bennyvw
Copy link
Copy Markdown
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
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

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.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@GerwinLaagland
Copy link
Copy Markdown
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
Copy Markdown

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
Copy Markdown

github-actions bot commented Mar 4, 2026

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 4, 2026

@github-actions
Copy link
Copy Markdown

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
Copy Markdown

github-actions bot commented Mar 4, 2026

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 4, 2026

1 similar comment
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 4, 2026

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 4, 2026

@github-actions
Copy link
Copy Markdown

@leeuwe
Copy link
Copy Markdown
Member

leeuwe commented Mar 26, 2026

@bennyvw Can you Resolve the conflicts and messages in this PR? Then we can go ahead merging

Copilot AI review requested due to automatic review settings March 26, 2026 16:38
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.

Corrected the link to the Mandatory Args section in the documentation.
Updated link to Mandatory Args documentation for clarity.
Copilot AI review requested due to automatic review settings March 26, 2026 16:57
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.

Corrected the link to the Mandatory Args section in the documentation.
Copilot AI review requested due to automatic review settings March 26, 2026 17:04
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

@GerwinLaagland
Copy link
Copy Markdown
Collaborator

Links and conflicts have been repaired. Please verify the revised sections.

@bennyvw
Copy link
Copy Markdown
Contributor Author

bennyvw commented Mar 26, 2026

My remark on the 'splitted' image is still open:
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.

I have been searching, but could not find a solution. Also GreenShot did not do the trick :/

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.

4 participants