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

Move module_utils out of subdir #23

Merged
merged 1 commit into from
Jun 16, 2020

Conversation

flowerysong
Copy link
Contributor

SUMMARY

ansible-collections/amazon.aws#26

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

@jillr
Copy link
Collaborator

jillr commented Apr 14, 2020

We should be consistent between the aws collections with where we import camel_dict_to_snake_dict and snake_dict_to_camel_dict from, I've already asked s-hertel for the backstory on importing that from ec2.py in the other PR so let's sort that question out there first.

@flowerysong
Copy link
Contributor Author

@jillr I was less concerned about cleaning that up in this PR, since it's not (strictly speaking) part of the directory flattening and is a less intrusive change.

@flowerysong flowerysong force-pushed the flatten-module-utils branch 3 times, most recently from 3b73a3d to 4c728bd Compare June 15, 2020 15:20
@jillr
Copy link
Collaborator

jillr commented Jun 16, 2020

@flowerysong I modified the test requirements file on you, apologies. I'd like to get this merged this week if possible, let me know if you don't have time to work on it and would like me to fix it up.

@flowerysong flowerysong changed the title [WIP] Move module_utils out of subdir Move module_utils out of subdir Jun 16, 2020
Copy link
Collaborator

@jillr jillr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - thanks!

@jillr jillr merged commit 10853d9 into ansible-collections:master Jun 16, 2020
@flowerysong flowerysong deleted the flatten-module-utils branch July 8, 2020 20:34
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 14, 2021
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 15, 2021
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 16, 2021
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 17, 2021
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request May 25, 2022
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request May 25, 2022
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request May 25, 2022
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request May 25, 2022
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants