Skip to content

Commit f8e7549

Browse files
committed
Merge remote-tracking branch 'upstream/develop' into develop
2 parents d233ac4 + f3b1555 commit f8e7549

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

49 files changed

+1260
-738
lines changed

.github/workflows/pythonapp.yml

Lines changed: 17 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@ on:
1515
jobs:
1616
build:
1717
runs-on: ubuntu-latest
18-
1918
steps:
2019
- uses: actions/checkout@v2
2120
- name: Set up Python 3.8
@@ -33,17 +32,31 @@ jobs:
3332
flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics
3433
# exit-zero treats all errors as warnings. The GitHub editor is 127 chars wide
3534
flake8 . --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics
35+
- name: Check formatting with black
36+
run: |
37+
# exit if any files still need formatting
38+
bash script/format --check
39+
continue-on-error: true
3640
- name: Test with pytest
3741
run: |
3842
pip install pytest
3943
python -m pytest
4044
- name: Run App
4145
run: |
42-
streamlit run src/app.py &
46+
PARAMETERS=./defaults/cypress.cfg streamlit run src/app.py &
4347
- name: Cypress
44-
uses: cypress-io/github-action@v1.23.2
48+
uses: cypress-io/github-action@v1
4549
with:
4650
working-directory: e2e
4751
wait-on: 'http://localhost:8000'
4852
wait-on-timeout: 300
49-
53+
- uses: actions/upload-artifact@v1
54+
if: failure()
55+
with:
56+
name: cypress-screenshots
57+
path: e2e/cypress/screenshots
58+
- uses: actions/upload-artifact@v1
59+
if: always()
60+
with:
61+
name: cypress-videos
62+
path: e2e/cypress/videos

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
# Editors
22
.idea/
3+
.vscode/settings.json
34

45
# Environments
56
.env

Dockerfile

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,13 @@
11
FROM python:3.7.7-slim-buster
2-
RUN mkdir /app
2+
ENV PARAMETERS=./defaults/webapp.cfg
33
WORKDIR /app
44
COPY README.md .
55
COPY setup.cfg .
66
COPY setup.py .
7-
COPY requirements.txt .
8-
# Creating an empty src dir is a (hopefully) temporary hack to improve layer caching and speed up image builds
9-
# todo fix once the Pipfile, setup.py, requirements.txt, pyprojec.toml build/dist story is figured out
10-
RUN mkdir src && pip install -q .
117
COPY .streamlit .streamlit
12-
COPY settings.cfg .
8+
COPY defaults defaults
139
COPY src src
10+
RUN pip install -q .
1411

1512
CMD ["streamlit", "run", "src/app.py"]
1613

Dockerfile.dash

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,11 +4,14 @@ WORKDIR /code
44

55
ARG PORT
66
ENV PORT $PORT
7+
ENV PARAMETERS=./defaults/webapp.cfg
78

8-
COPY requirements.txt requirements.txt
9-
RUN pip install -r requirements.txt
10-
9+
COPY README.md .
10+
COPY setup.py .
11+
COPY setup.cfg .
1112
COPY src src
13+
COPY defaults ./src/defaults
14+
RUN pip install -q .
1215

1316
# EXPOSE $PORT
1417

Pipfile

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,12 +4,13 @@ url = "https://pypi.org/simple"
44
verify_ssl = true
55

66
[dev-packages]
7+
black = "==19.10b0"
8+
pytest = "*"
79

810
[packages]
911
streamlit = "*"
1012
pandas = "*"
1113
numpy = "*"
12-
pytest = "*"
1314
altair = "*"
1415
dash = "*"
1516
dash-bootstrap-components = "*"
@@ -18,3 +19,6 @@ PyYAML = "*"
1819

1920
[requires]
2021
python_version = "3.7"
22+
23+
[pipenv]
24+
allow_prereleases = true

0 commit comments

Comments
 (0)