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

Fix deprecations for plugins and role entrypoints #156

Merged
merged 5 commits into from
Jun 9, 2023

Conversation

felixfontein
Copy link
Collaborator

  • Fix indent of 'Why' and 'Alternative' texts for deprecations. If these texts happen to be multiple lines long, the other lines must be indented.
  • Allow role entrypoint deprecations without explicitly specifying removed_from_collection (which is the collection the role is part of anyway).

@felixfontein felixfontein added the backport-1 Automatically create backport to stable-1 branch label Jun 7, 2023
Co-authored-by: Brian Scholer <1260690+briantist@users.noreply.github.com>
@felixfontein felixfontein merged commit 2d3c65e into ansible-community:main Jun 9, 2023
@patchback
Copy link

patchback bot commented Jun 9, 2023

Backport to stable-1: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-1/2d3c65e2d3d5af7bd5e9c6af41b3007c6f7161c5/pr-156

Backported as #158

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

@felixfontein felixfontein deleted the indent branch June 9, 2023 04:13
patchback bot pushed a commit that referenced this pull request Jun 9, 2023
* Fix indent of 'Why' and 'Alternative' texts.

* Add tests.

* Allow role entrypoint deprecations without having to specify the collection the role is removed from.

* Fix YAML.

* Fix typo.

Co-authored-by: Brian Scholer <1260690+briantist@users.noreply.github.com>

---------

Co-authored-by: Brian Scholer <1260690+briantist@users.noreply.github.com>
(cherry picked from commit 2d3c65e)
@felixfontein
Copy link
Collaborator Author

@briantist thanks a lot!

felixfontein added a commit that referenced this pull request Jun 9, 2023
…d role entrypoints (#158)

* Fix deprecations for plugins and role entrypoints (#156)

* Fix indent of 'Why' and 'Alternative' texts.

* Add tests.

* Allow role entrypoint deprecations without having to specify the collection the role is removed from.

* Fix YAML.

* Fix typo.

Co-authored-by: Brian Scholer <1260690+briantist@users.noreply.github.com>

---------

Co-authored-by: Brian Scholer <1260690+briantist@users.noreply.github.com>
(cherry picked from commit 2d3c65e)

* Ignore linting error.

---------

Co-authored-by: Felix Fontein <felix@fontein.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-1 Automatically create backport to stable-1 branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants