Bartłomiej Hirsz

Results 84 issues of Bartłomiej Hirsz

From https://github.com/robotframework/robotframework/issues/4371 1) See if any rules needs update (like duplicated import name) 2) Add deprecation warning for ``WITH NAME`` starting from RF 5.1

From https://github.com/robotframework/robotframework/issues/4373 This require: 1) updating our visitors (so tags from Keywords Tags are visited and also setting related rules like naming ones does work on new setting) 2) refactoring...

If there was fatal issue in RF parsing module and somebody uses Robocop with older (bugged) version, Robocop will fail with Robot Framework stack trace. For example empty arguments cause...

enhancement

Robot Framework 5.0 introduce new syntax: TRY EXCEPT blocks. We need to handle them. Things to consider: 1. ~~Other rules should be able to access TRY, EXCEPT, FINALLY, ELSE blocks.~~...

Handle BREAK and CONTINUE. Possible rules (update old or create new): 1. Do not name keyword after reserved words (#570) 2. Do not use BREAK, CONTINUE outside FOR, WHILE or...

rule

Right now we are recognizing file type in this code: https://github.com/MarketSquare/robotframework-robocop/blob/af807515d7f160f6673f726f3b7cd33bca092a21/robocop/run.py#L93 (basically, we check if the file imports something, and if we're importing file that's being scanned by us -...

rule

Related #524 If Robocop auto detect file as resource type, but it will contain `*** Test Case / Task ***` section it will be reported as parsing error. It should...

Originated from Slack question: https://robotframework.slack.com/archives/C0K0240NL/p1641402075295600 Overwriting automatic/reserved variables may bring unexpected results so it'll would be good to warn when it happens. https://robotframework.org/robotframework/latest/RobotFrameworkUserGuide.html#automatic-variables

rule

See https://robotframework.org/robotframework/latest/RobotFrameworkUserGuide.html#id605 . We need to verify if it doesn't break anything in Robocop and all issues are properly reported. Should we suggest using Task X variant if there is...

Empty test template can be used to overwrite suite test template. But if a) there is no test template in suite settigns b) we're using [Template] without more explicit NONE...

rule