search

asyrjasalo / RESTinstance

Robot Framework library for RESTful JSON APIs
https://asyrjasalo.github.io/RESTinstance
GNU Lesser General Public License v3.0
203 stars 84 forks source link

Set log level also part of RF Builtin #143

Closed Harm10 closed 1 year ago

Harm10 commented 1 year ago

I am getting a warning when using Set log level command: [ WARN ] Keyword 'Set Log Level' found both from a custom test library 'REST' and a standard library 'BuiltIn'. The custom keyword is used. To select explicitly, and to get rid of this warning, use either 'REST.Set Log Level' or 'BuiltIn.Set Log Level'. [ WARN ] Unrecognized log level 'NONE'. Using default log level 'WARN'.

Obviously I want to use the Builtin version. Can you please change this command within RESTinstance so this problem will disappear?

asyrjasalo commented 1 year ago

Hi,

Our code of conduct is that we do not change keywords.

Please report the issue in the builtin library repo.

BR, Anssi Lead Developer, Robot Framework

Harm10 commented 1 year ago

@asyrjasalo That is turning the world around! Builtin is core of RF so in my view no package should use keywords that are defined in there. So please reopen this issue?

asyrjasalo commented 1 year ago

Hi,

Our code of conduct is that we target the library for those people who are capable in turning the world around.

There is possibly some hint in the library name.

Hope this clarifies.

BR, Anssi Lead Developer, Robot Framework

Harm10 commented 1 year ago

Sorry to say so: I think this is a stupid answer. So in creating a package to be used in Robot Framework you think it is normal to reuse a core key word? Do you think users will inactivate Builtin? I guess not.

So you will probably understand I will uninstall your package and will search for another package that works together with Robot Framework as all others do!