-
Notifications
You must be signed in to change notification settings - Fork 346
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
cloudwatchevent_rule should return false when there is no change done to the rule #1589
Merged
softwarefactory-project-zuul
merged 22 commits into
ansible-collections:main
from
taehopark32:branch_new
Jul 4, 2023
Merged
Changes from 21 commits
Commits
Show all changes
22 commits
Select commit
Hold shift + click to select a range
fd16836
Changes to the _target_to_put in cloudwatchevent_rule
taehopark32 d3887d1
Changes made to method (_target_to_put) in cloudwatcheevent_ruler
taehopark32 d940d0a
Changes to method (_target_to_put) in cloudwatchevent_ruler
taehopark32 b974349
Changes to method (_target_to_put) in cloudwatchevent_ruler
taehopark32 c950f50
test
taehopark32 1a64051
Update README.md
taehopark32 0ef87b2
deleted the keys
taehopark32 98a405a
added fragmens and tests
taehopark32 2e19dd4
changes to the target (for accurate discription) and cloudwatchevent_…
taehopark32 3bc5d03
change to the test tasks in cloudwatchevent_rule for Assert that no c…
taehopark32 5b9f256
changes to the Assert that no changes were made to the rule
taehopark32 5c86663
Changes to the test
taehopark32 15a646b
Changes to the idempotent
taehopark32 bc356b9
Added quotes
taehopark32 d3a833d
removed import q
taehopark32 58948b9
added another condition where input_transformer could be None
taehopark32 38583bd
remove a file
taehopark32 44c361e
fixed the quote problem
taehopark32 0786e93
changes to the suggestion
taehopark32 c8ac229
fix trailing space in integration cloudwatchevent_rule
taehopark32 807ad1e
used tox to structure
taehopark32 0d56a8c
changed description to be more user friendly
taehopark32 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,3 @@ | ||
--- | ||
bugfixes: | ||
- cloudwatchevent_rule - add scrub none inside the _targets_to_put method (https://github.com/ansible-collections/amazon.aws/pull/1589)." | ||
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
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
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changelog should describe what the end user needs to know, functionaly, about the change, rather than the implementation details of the code. Something like
"Fixes changed status to report False when no change has been made. The module had incorrectly always reported a change."
But make sure it describes whatever scenario the wrong change status would have occurred for, if it wasn't always.