Update automatetheautomation.md #94
Merged
Conversation
This file contains hidden or 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
Re-wording on line 9
cidrbl0ck
requested review from
adonisgarciac,
ivarmu and
silvinux
as code owners
djdanielsson
reviewed
Apr 1, 2025
|
Ah sounds like a deeper rewrite than what I looked at. |
sean-m-sullivan
approved these changes
Apr 3, 2025
djdanielsson
requested changes
Apr 3, 2025
|
FWIW I'd be willing to give it a shot, but TBH I don't quite know dispatchers role in things. |
|
the dispatchers role is calling all the other roles to apply the configuration in the correct order |
sean-m-sullivan
approved these changes
Aug 29, 2025
djdanielsson
approved these changes
Aug 29, 2025
piergiovannimanfrin
pushed a commit
to piergiovannimanfrin/aap_configuration_extended
that referenced
this pull request
Sep 2, 2025
Re-wording on line 9
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
Suggestion cannot be applied right now. Please check back later.
Re-wording on line 9
What does this PR do?
I just re-wored the end of line 9 a bit to be more accurate based on what's presented below line 9. It lists 4 roles instead of 3. And comprised of reads better than compose by.
How should this be tested?
Is there a relevant Issue open for this?
resolves #[number]
Other Relevant info, PRs, etc