Skip to content
This repository has been archived by the owner on Sep 9, 2022. It is now read-only.

Set force_destroy: false for IAM users #267

Merged
merged 1 commit into from
Sep 7, 2016
Merged

Set force_destroy: false for IAM users #267

merged 1 commit into from
Sep 7, 2016

Conversation

raylu
Copy link
Contributor

@raylu raylu commented Sep 2, 2016

Otherwise you get output like this:

~ aws_iam_user.raylu
    force_destroy: "" => "false"

@coveralls
Copy link

coveralls commented Sep 2, 2016 •
edited
Loading

Coverage Status

Coverage remained the same at 100.0% when pulling d5eab7c on stripe:iamu-force_destroy into 9fb0d72 on dtan4:master.

@dtan4
Copy link
Owner

dtan4 commented Sep 5, 2016

Could you update the test iam_user_spec.rb too?

@raylu raylu force-pushed the iamu-force_destroy branch from d5eab7c to af90499 Compare September 6, 2016 18:48
@coveralls
Copy link

coveralls commented Sep 6, 2016 •
edited
Loading

Coverage Status

Coverage remained the same at 100.0% when pulling af90499 on stripe:iamu-force_destroy into 9fb0d72 on dtan4:master.

@raylu
Copy link
Contributor Author

raylu commented Sep 6, 2016

Sorry, didn't see the test failure. Amended with a fix!

@dtan4
Copy link
Owner

dtan4 commented Sep 7, 2016

LGTM 👍 Thanks ❗

@dtan4 dtan4 merged commit 3739fa1 into dtan4:master Sep 7, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@raylu @coveralls @dtan4 @raylu-stripe