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.
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/ |
|
🚀 Preview deployed to https://robotframework-RFCP-syllabus-pr-80.surge.sh/robotframework-RFCP-syllabus/ |
|
@bennyvw Can you Resolve the conflicts and messages in this PR? Then we can go ahead merging |
Corrected the link to the Mandatory Args section in the documentation.
Updated link to Mandatory Args documentation for clarity.
Corrected the link to the Mandatory Args section in the documentation.
|
🚀 Preview deployed to https://robotframework-RFCP-syllabus-pr-80.surge.sh/robotframework-RFCP-syllabus/ |
|
Links and conflicts have been repaired. Please verify the revised sections. |
|
My remark on the 'splitted' image is still open: I have been searching, but could not find a solution. Also GreenShot did not do the trick :/ |
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.