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

[WIP] meraki_mr_l3_firewall - Enable check mode #54749

Open
wants to merge 2 commits into
base: devel
from

Conversation

Projects
None yet
2 participants
@kbreit
Copy link
Contributor

commented Apr 3, 2019

SUMMARY

Enable check mode.

Relates to #53597.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

meraki_mr_l3_firewall

@ansibot

This comment has been minimized.

Copy link
Contributor

commented Apr 3, 2019

The test ansible-test sanity --test pep8 [explain] failed with 5 errors:

lib/ansible/modules/network/meraki/meraki_mr_l3_firewall.py:276:53: E226 missing whitespace around arithmetic operator
lib/ansible/modules/network/meraki/meraki_mr_l3_firewall.py:277:53: E226 missing whitespace around arithmetic operator
lib/ansible/modules/network/meraki/meraki_mr_l3_firewall.py:279:41: E226 missing whitespace around arithmetic operator
lib/ansible/modules/network/meraki/meraki_mr_l3_firewall.py:281:41: E226 missing whitespace around arithmetic operator
lib/ansible/modules/network/meraki/meraki_mr_l3_firewall.py:295:5: E303 too many blank lines (2)

click here for bot help

@ansibot

This comment has been minimized.

Copy link
Contributor

commented Apr 3, 2019

@ansibot ansibot removed the ci_verified label Apr 3, 2019

@ansibot ansibot added the stale_ci label Apr 11, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.