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.

Sign up for GitHub

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

Jump to bottom

#30 fix for filtering accepter subnets using accepter_subnet_tags #31

Closed
wants to merge 1 commit into from
Closed

#30 fix for filtering accepter subnets using accepter_subnet_tags #31

wants to merge 1 commit into from

Conversation

maxvasylets
Copy link

what

This is the fix for #30

why

accepter_subnet_tags doesn't have any effect

@maxvasylets maxvasylets requested a review from a team as a code owner December 2, 2020 15:53
@maxvasylets maxvasylets requested review from jamengual and danjbh and removed request for a team December 2, 2020 15:53
@Nuru Nuru mentioned this pull request Feb 6, 2021
Open
@maxvasylets
Copy link
Author

Could anyone help me understand how can I run the required checks?

@aknysh aknysh mentioned this pull request Mar 20, 2021
Merged
@aknysh aknysh closed this in #44 Mar 23, 2021
@aknysh
Copy link
Member

aknysh commented Mar 23, 2021

@maxvasylets
Thanks a lot for your PR.
Your changes were added in the latest release https://github.com/cloudposse/terraform-aws-vpc-peering-multi-account/releases/tag/0.15.0

@Nuru Nuru mentioned this pull request Apr 9, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@philippegomes philippegomes philippegomes approved these changes

@jamengual jamengual Awaiting requested review from jamengual

@danjbh danjbh Awaiting requested review from danjbh

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@maxvasylets @aknysh @philippegomes