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

removed duplicated code #53

Merged
merged 1 commit into from
May 12, 2022
Merged

removed duplicated code #53

merged 1 commit into from
May 12, 2022

Conversation

brunoscota
Copy link
Contributor

@brunoscota brunoscota commented Apr 29, 2022

Description

Im receiving an error to deploy a not_statement for ip_set_reference because i guess theres a duplicated code. So in my PR i removed the duplicated block, please reviewed if I did it right

YOu can see there is a duplicated block on line 1347 and 1413 of main.tffile. After removed the duplicated code my code run perfectly.

Issue reported here: #52

@brunoscota
Copy link
Contributor Author

sorry if I did any mistake but its very important to me. And this error its blocking me.

Copy link
Contributor

@Ohid25 Ohid25 left a comment

Choose a reason for hiding this comment

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

Nice find!

LGTM!

@Ohid25 Ohid25 merged commit e7fb788 into umotif-public:main May 12, 2022
@Ohid25 Ohid25 added the bug Something isn't working label May 12, 2022
@Ohid25
Copy link
Contributor

Ohid25 commented May 12, 2022

Fixed with 3.7.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Development

Successfully merging this pull request may close these issues.

2 participants