Skip to content

Commit

Permalink
Merge pull request #815 from sanger/develop
Browse files Browse the repository at this point in the history
Develop into master
  • Loading branch information
yoldas authored Aug 11, 2023
2 parents 0cd2db1 + 27f0423 commit 1b8fbe8
Show file tree
Hide file tree
Showing 7 changed files with 574 additions and 593 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/automated_release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ jobs:
runs-on: ubuntu-latest
services:
mysql:
image: mysql:5.7
image: mysql:8.0
ports:
- 3306:3306
env:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ jobs:
runs-on: ubuntu-latest
services:
mysql:
image: mysql:5.7
image: mysql:8.0
ports:
- 3306:3306
env:
Expand Down
2 changes: 1 addition & 1 deletion .release-version
Original file line number Diff line number Diff line change
@@ -1 +1 @@
2.34.0
2.35.0
4 changes: 2 additions & 2 deletions Pipfile
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ colorlog = "~=6.7"
eve = "~=2.1"
flask-apscheduler = "~=1.12"
flask-cors = "~=4.0"
gunicorn = "~=20.0"
gunicorn = "~=21.2"
openpyxl = "~=3.1"
pandas = "~=2.0"
pika = "~=1.3"
Expand All @@ -30,7 +30,7 @@ python-dotenv = "~=1.0"
requests = "~=2.31"
slackclient = "~=2.9"
sqlalchemy = "~=2.0"
pymongo = "~=4.4.0"
pymongo = "~=4.4.1"

[requires]
python_version = "3.8"
Expand Down
1,142 changes: 563 additions & 579 deletions Pipfile.lock

Large diffs are not rendered by default.

6 changes: 3 additions & 3 deletions dependencies/docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,11 @@ version: "3.8"
services:
# A MySQL service as close to production as currently possible
# To run it as a standalone container:
# docker run --name mysql-5.7 -e MYSQL_ALLOW_EMPTY_PASSWORD=yes -d -p 3306:3306 mysql:5.7
mysql-5.7:
# docker run --name mysql-8.0 -e MYSQL_ALLOW_EMPTY_PASSWORD=yes -d -p 3306:3306 mysql:8.0
mysql-8.0:
# Use the following on M1; for Docker Rosetta support on macOS Ventura.
# platform: linux/amd64
image: mysql:5.7
image: mysql:8.0
ports:
- "3306:3306"
environment:
Expand Down
9 changes: 3 additions & 6 deletions lighthouse/validator.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,7 @@

from lighthouse.classes.beckman import Beckman
from lighthouse.classes.biosero import Biosero
from lighthouse.constants.error_messages import (
ERROR_PLATE_SPECS_EMPTY_LIST,
ERROR_PLATE_SPECS_INVALID_FORMAT,
)
from lighthouse.constants.error_messages import ERROR_PLATE_SPECS_EMPTY_LIST, ERROR_PLATE_SPECS_INVALID_FORMAT
from lighthouse.constants.fields import FIELD_EVENT_RUN_ID

logger = logging.getLogger(__name__)
Expand Down Expand Up @@ -95,7 +92,7 @@ def _check_with_validate_cptd_plate_specs(self, field, value):
if len(value) == 0:
self._error(field, ERROR_PLATE_SPECS_EMPTY_LIST)

if not all([type(ps) == list and len(ps) == 2 for ps in value]) or not all(
[type(s) == int for ps in value for s in ps]
if not all([type(ps) is list and len(ps) == 2 for ps in value]) or not all(
[type(s) is int for ps in value for s in ps]
):
self._error(field, ERROR_PLATE_SPECS_INVALID_FORMAT)

0 comments on commit 1b8fbe8

Please sign in to comment.