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.

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

10.0.0: CmdRunner: remove deprecated feature #8928

Merged

Conversation

russoz
Copy link
Collaborator

@russoz russoz commented Sep 26, 2024

SUMMARY

Remove deprecated feature from CmdRunner

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

plugins/module_utils/cmd_runner.py

@russoz russoz changed the title [WIP] 10.0.0.0: Cmd Runner: remove deprecated feature [WIP] 10.0.0.0: CmdRunner: remove deprecated feature Sep 26, 2024
@ansibullbot ansibullbot added WIP Work in progress feature This issue/PR relates to a feature request module_utils module_utils plugins plugin (any type) tests tests unit tests/unit labels Sep 26, 2024
@felixfontein felixfontein added the breaking_change This PR contains a breaking change that MUST NOT be backported label Sep 26, 2024
@felixfontein felixfontein marked this pull request as draft September 26, 2024 06:38
@felixfontein felixfontein changed the title [WIP] 10.0.0.0: CmdRunner: remove deprecated feature [WIP] 10.0.0: CmdRunner: remove deprecated feature Sep 26, 2024
@ansibullbot ansibullbot added the integration tests/integration label Sep 26, 2024
@russoz russoz changed the title [WIP] 10.0.0: CmdRunner: remove deprecated feature 10.0.0: CmdRunner: remove deprecated feature Sep 26, 2024
@russoz russoz requested a review from felixfontein September 26, 2024 08:44
@russoz russoz marked this pull request as ready for review September 26, 2024 08:45
@ansibullbot ansibullbot removed the WIP Work in progress label Sep 26, 2024
@felixfontein
Copy link
Collaborator

The change looks good, but it needs a changelog fragment :)

@russoz
Copy link
Collaborator Author

russoz commented Sep 26, 2024

That was fast

@felixfontein
Copy link
Collaborator

I was waiting for CI to pass to take another look, and then I saw you asked for a review anyway ;)

@felixfontein felixfontein merged commit 002f137 into ansible-collections:main Sep 26, 2024
150 checks passed
@felixfontein
Copy link
Collaborator

@russoz thanks!

@russoz russoz deleted the cmd-runner-deprecation branch September 26, 2024 09:02
aioue pushed a commit to aioue/community.general that referenced this pull request Oct 1, 2024
* remove deprecated feature

* remove deprecated feature from tests as well

* Adjust the integration test

* add changelog frag
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking_change This PR contains a breaking change that MUST NOT be backported feature This issue/PR relates to a feature request integration tests/integration module_utils module_utils plugins plugin (any type) tests tests unit tests/unit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants