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
Add a dynamic version of blocks. #72397
Open
estheruary
wants to merge
4
commits into
ansible:devel
Choose a base branch
from
estheruary:add_dynamic_version_of_block
base: devel
Could not load branches
Branch not found: {{ refName }}
Could not load tags
Nothing to show
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
+399
−1
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
4b0acc8
to
3119fa9
Compare
The test
The test
The test
|
0bdaf25
to
0532e70
Compare
0532e70
to
53cf1b6
Compare
@estheruary this PR contains the following merge commits: Please rebase your branch to remove these commits. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
affects_2.11
feature
This issue/PR relates to a feature request.
merge_commit
This PR contains at least one merge commit. Please resolve!
module
This issue/PR relates to a module.
needs_rebase
https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html
needs_revision
This PR fails CI tests or a maintainer has requested a review/revision of the PR.
new_module
This PR includes a new module.
new_plugin
This PR includes a new plugin.
pre_azp
This PR was last tested before migration to Azure Pipelines.
support:core
This issue/PR relates to code supported by the Ansible Engineering Team.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
SUMMARY
Adds a new keyword/module called
do
which is the dynamic version ofblock
. Among other things, this allows you to loop over multiple tasks without including an external file.We have
include
as the dynamic version ofimport
. How aboutdo
as the dynamic version ofblock
?ISSUE TYPE
Spiritually implements #13262
COMPONENT NAME
do
ADDITIONAL INFORMATION
Right now the feature works but obviously needs some testing and documentation. Figured I'd hold off on the tedious parts until I find out if this is something you folks actually want upstream.
The name is inspired by how Ruby handles passing blocks.