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

Improve check_snake_case_filename check in msftidy #10753

Merged
merged 1 commit into from Oct 5, 2018

Conversation

Projects
None yet
2 participants
@wvu-r7
Contributor

wvu-r7 commented Oct 5, 2018

Improve check_snake_case_filename check in msftidy
We also remove the separator, since the file is basenamed.

@busterb busterb merged commit 05ac387 into rapid7:master Oct 5, 2018

3 checks passed

Metasploit Automation - Sanity Test Execution Successfully completed all tests.
Details
Metasploit Automation - Test Execution Successfully completed all tests.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

busterb added a commit that referenced this pull request Oct 5, 2018

@busterb

This comment has been minimized.

Show comment
Hide comment
@busterb

busterb Oct 5, 2018

Contributor

Leaving this without msf5 since it'll be good to maintain it across branches. Thanks!

Contributor

busterb commented Oct 5, 2018

Leaving this without msf5 since it'll be good to maintain it across branches. Thanks!

msjenkins-r7 added a commit that referenced this pull request Oct 5, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment