ome / omero-scripts

Core OMERO Scripts
https://pypi.org/project/omero-scripts/
12 stars 32 forks source link

Fix flake8 #149

Closed joshmoore closed 5 years ago

joshmoore commented 5 years ago

Last minute fix.

see: https://travis-ci.org/openmicroscopy/openmicroscopy/builds/554133967

joshmoore commented 5 years ago
waiting on port: #1
waiting on port: #2
waiting on port: #3
waiting on port: #4
waiting on port: #5
waiting on port: #6
waiting on port: #7
waiting on port: #8
waiting on port: #9
waiting on port: #10
Sleeping for 30
Login attempt 1 failed. Trying again in 2 seconds
Login attempt 2 failed. Trying again in 3 seconds
Login attempt 3 failed. Trying again in 4 seconds
+ '[' -z '' ']'
+ export ICE_HOME=/usr/share/Ice-3.6.4
+ ICE_HOME=/usr/share/Ice-3.6.4
+ '[' 1 -eq 0 ']'
+ for arg in "$@"
+ case $arg in
+ py_flake8 java_build java_test py_test
+ flake8 .
./components/tools/OmeroPy/scripts/test/integration/test_util_scripts.py:133:13: E126 continuation line over-indented for hanging indent
Stopping openmicroscopy_db_1    ... 
Stopping openmicroscopy_test_1  ... 
Stopping openmicroscopy_omero_1 ... 
oving openmicroscopy_db_1    ... 
Removing openmicroscopy_test_1  ... 
Removing openmicroscopy_omero_1 ... 

Does this mean that the tests in this repo aren't being flake8'd by travis but they are in the main repo?

cc: @sbesson

joshmoore commented 5 years ago

Seems to be the case since omero-test-infra doesn't copy the script tests:

/opt/omero-test-infra $cat scripts-copy
#!/bin/bash

set -e
set -u
set -x

rm -rf $OMERO_DIST/lib/scripts/omero

cd $TARGET
cp -r omero $OMERO_DIST/lib/scripts/