Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

OperatingSystem: Deprecate all keywords related to Start Process #2158

Closed
tarand opened this issue Sep 24, 2015 · 3 comments
Closed

OperatingSystem: Deprecate all keywords related to Start Process #2158

tarand opened this issue Sep 24, 2015 · 3 comments
Assignees
@pekkaklarck
Labels
deprecation enhancement priority: low
Milestone
2.9.2

Comments

@tarand
Copy link

tarand commented Sep 24, 2015

I'm migrating to Robot 2.9. During run of my testcases a deprecation warning for OS.Start Process pops up.
But there is no deprecation warning for "Stop Process". This is very dangerous, as it could lead to users using Process.Start Process but still use OS.Stop Process.
Please also show a deprecation warning for OS.Stop Process!
@pekkaklarck
Copy link
Member

We only added actual deprecation warning to OS.Start Process to avoid showing too may warnings. I can see that loudly deprecating all makes it easier to see places where old keywords are used.

@pekkaklarck
Copy link
Member

The issue about deprecation OS.Start Process is #1773.

@pekkaklarck pekkaklarck added this to the 2.9.2 milestone Sep 24, 2015
@pekkaklarck pekkaklarck self-assigned this Sep 24, 2015
@pekkaklarck pekkaklarck changed the title Deprecation warning for "Stop Process" needed OperatingSystem: Deprecate all keywords related to Start Process Sep 24, 2015
@pekkaklarck
Copy link
Member

These keywords were removed in RF 3.0 (#2181).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pekkaklarck pekkaklarck

Labels
deprecation enhancement priority: low
Projects
None yet
Milestone
2.9.2
Development

No branches or pull requests
2 participants
@pekkaklarck @tarand