Skip to content

Conversation

@HolQue
Copy link
Collaborator

@HolQue HolQue commented Aug 14, 2025

Reference:
#421 (comment)

@HolQue HolQue requested review from Copilot and namsonx August 14, 2025 10:24
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

This PR extends the documentation to clarify how special characters should be handled within key names in JSONP format files. The update addresses potential confusion about bracket usage and character escaping in JSON key names.

Key changes:

  • Added a new section explaining restrictions on special characters in JSONP key names
  • Documented the problematic use of syntax brackets ([, ], {, }, <, >) in key names
  • Provided examples showing proper handling of other special characters and string escaping

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

causes:
\begin{pythonlog}
Error: 'Invalid expression found: '${[}' - The brackets mismatch!!!'!
Copy link

Copilot AI Aug 14, 2025

Choose a reason for hiding this comment

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

There are too many exclamation marks in the error message. It should be either 'The brackets mismatch!' or 'The brackets mismatch!!!' but not both.

Suggested change
Error: 'Invalid expression found: '${[}' - The brackets mismatch!!!'!
Error: 'Invalid expression found: '${[}' - The brackets mismatch!'

Copilot uses AI. Check for mistakes.
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Hi Son,

maybe a small maintenance activity for you.

@test-fullautomation test-fullautomation added this to the 0.14.2 milestone Aug 14, 2025
@test-fullautomation test-fullautomation merged commit d81303a into namsonx/task/stabi_branch Aug 14, 2025
1 check passed
@github-project-automation github-project-automation bot moved this from Open to Done in RobotFramework AIO Aug 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

3 participants