Updated explanation for Keyword Matching (3.3.2) and examples for Free Named Arguments (2.5.1.2)#80
Updated explanation for Keyword Matching (3.3.2) and examples for Free Named Arguments (2.5.1.2)#80
Conversation
There was a problem hiding this comment.
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 ProcesstoSet 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 Dictionarykeyword 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. |
There was a problem hiding this comment.
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.
| 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. |
|
|
||
| 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.
This comment was marked as resolved.
Sorry, something went wrong.
There was a problem hiding this comment.
The link chapter-03/03_user_keyword.md led to a 404..
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
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 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:
In this PR, the script shows a diff for How to fixPlease regenerate the generated files locally and commit the results: python tools/gen_numbering.pyThen commit whatever files changed (often Once the regenerated output is committed, the CI check should pass. (Thanks ChatGPT for helping me out. 😅) |
|
🚀 Preview deployed to https://robotframework-RFCP-syllabus-pr-80.surge.sh/robotframework-RFCP-syllabus/ |
|
🚀 Preview deployed to https://robotframework-RFCP-syllabus-pr-80.surge.sh/robotframework-RFCP-syllabus/ |
|
🚀 Preview deployed to https://robotframework-RFCP-syllabus-pr-80.surge.sh/robotframework-RFCP-syllabus/ |
|
🚀 Preview deployed to https://robotframework-RFCP-syllabus-pr-80.surge.sh/robotframework-RFCP-syllabus/ |
|
🚀 Preview deployed to https://robotframework-RFCP-syllabus-pr-80.surge.sh/robotframework-RFCP-syllabus/ |
|
🚀 Preview deployed to https://robotframework-RFCP-syllabus-pr-80.surge.sh/robotframework-RFCP-syllabus/ |
1 similar comment
|
🚀 Preview deployed to https://robotframework-RFCP-syllabus-pr-80.surge.sh/robotframework-RFCP-syllabus/ |
|
🚀 Preview deployed to https://robotframework-RFCP-syllabus-pr-80.surge.sh/robotframework-RFCP-syllabus/ |
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
impact:majororimpact:minorMajor change checklist
Issues
Herewith issues and are solved.