-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update Dockerfile #402
Update Dockerfile #402
Conversation
This fixes the problem with limit when it is not a number
fix: update dparse to 0.5.2
BREAKING CHANGE: Remove support for json configuration files
Tests/python icatv1
Tests/reorganise tests#375
Conflicts: poetry.lock
…rator#369 Feature/icat backend generator#369
Codecov ReportBase: 94.47% // Head: 42.30% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## k8s-deployment #402 +/- ##
===================================================
- Coverage 94.47% 42.30% -52.18%
===================================================
Files 39 39
Lines 3114 3319 +205
Branches 311 310 -1
===================================================
- Hits 2942 1404 -1538
- Misses 144 1907 +1763
+ Partials 28 8 -20
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
Dockerfile
Outdated
COPY datagateway_api ./datagateway_api | ||
|
||
# Install & run stage | ||
FROM python:3-alpine |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Best to pin the version of Python to prevent unexpected breakages in future.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Resolved in dfb54ba
Merges main into this branch. Only the last commit needs to be reviewed.
python:3-alpine
base imageconfig.yaml
from environment variables