Skip to content

Commit 33f2a89

Browse files
committed
Add python linter and formatter
1 parent 90f18e4 commit 33f2a89

File tree

5 files changed

+36
-0
lines changed

5 files changed

+36
-0
lines changed

.flake8

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
[flake8]
2+
exclude =
3+
./tools/*,
4+
./venv/*,
5+
./twiml/*,
6+
./guides/*,
7+
./testable_snippets/*,
8+
*/testable_snippets/*
9+
10+
ignore =
11+
# D100: Missing docstring in public module
12+
D100,
13+
# H306: imports not in alphabetical order (time, os)
14+
H306,

.style.yapf

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
[style]
2+
dedent_closing_brackets = True
3+
join_multiple_lines = False

install_travis_dependencies.sh

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@ sudo apt-get install -y --force-yes gradle
1616
if [[ $languages == *"python"* ]]; then
1717
sudo apt-get install -y --force-yes python-dev python-pip
1818
sudo pip install --upgrade pip wheel virtualenv virtualenvwrapper --quiet
19+
sudo pip install -r requirements.txt
1920
fi
2021

2122
if [[ $languages == *"php"* ]]; then

requirements.txt

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
configparser==3.5.0
2+
enum34==1.1.6
3+
flake8==3.4.1
4+
mccabe==0.6.1
5+
pycodestyle==2.3.1
6+
pyflakes==1.5.0
7+
yapf==0.16.3

tools/snippet-testing/language_handler/python.rb

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,17 @@ module LanguageHandler
44
class Python < BaseHandler
55
LANG_CNAME = 'py'.freeze
66

7+
def self.run_before_test(directory)
8+
dirs = ['./venv/*', './guides/*', './testable_snippets/*']
9+
exclude = dirs.map{|dir| "--exclude='#{dir}'"}
10+
Dir.chdir(directory) do
11+
output = `yapf #{exclude.join(' ')} -i -r . && flake8`
12+
if $? != 0
13+
abort(output)
14+
end
15+
end
16+
end
17+
718
private
819

920
def execute_command(file)

0 commit comments

Comments
 (0)