Skip to content

Test violations#2

Open
Zaprogramiacz wants to merge 1 commit into
masterfrom
test
Open

Test violations#2
Zaprogramiacz wants to merge 1 commit into
masterfrom
test

Conversation

@Zaprogramiacz

Copy link
Copy Markdown
Contributor

No description provided.

@Zaprogramiacz Zaprogramiacz self-assigned this Feb 14, 2020
line = text_violation.split(":")[1]
return Violation(message=message, file_name=file_name, line=line)


Copy link
Copy Markdown

Choose a reason for hiding this comment

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

  • ⚠️ W391 blank line at end of file

Comment thread danger_flake8/plugin.py
@@ -1,4 +1,5 @@
import subprocess
import profile

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

  • ⚠️ F401 'profile' imported but unused

Comment thread tests/test_plugin.py
assert danger.results.warnings == []


@pytest.mark.parametrize("created_files", [["tests/test_violation.py"]])

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

  • ⚠️ E302 expected 2 blank lines, found 1

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