-
Notifications
You must be signed in to change notification settings - Fork 0
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
Ids catalogue properties #515 #516
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #516 +/- ##
===========================================
+ Coverage 93.98% 94.06% +0.07%
===========================================
Files 62 63 +1
Lines 17700 17782 +82
Branches 2389 2076 -313
===========================================
+ Hits 16636 16726 +90
+ Misses 1058 1054 -4
+ Partials 6 2 -4 ☔ View full report in Codecov by Sentry. |
.github/workflows/ci-build.yml
Outdated
@@ -82,7 +82,7 @@ jobs: | |||
MONGO_INITDB_ROOT_PASSWORD: example | |||
|
|||
ims-api: | |||
image: harbor.stfc.ac.uk/inventory-management-system/ims-api:develop | |||
image: harbor.stfc.ac.uk/inventory-management-system/ims-api:pr-240 |
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.
image: harbor.stfc.ac.uk/inventory-management-system/ims-api:pr-240 | |
image: harbor.stfc.ac.uk/inventory-management-system/ims-api:develop |
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.
Just a couple minor comments but LGTM.
Co-authored-by: Joel Davies <[email protected]>
Co-authored-by: Joel Davies <[email protected]>
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.
LGTM
* develop: Update dependency eslint-plugin-cypress to v3 Update tanstack-query monorepo to v5.31.0 Update actions/setup-python action to v5.1.0 Update actions/checkout digest Fix connection after changing backend docker compose to host networking #499 Use develop branch for dev cli script Update dependency material-react-table to v2.13.0 Use dev_cli script from api to setup the database for e2e api tests #499
the e2e tests are failing because the backend hasn't been merged in |
56b61ef
to
6a0a71f
Compare
Description
Testing instructions
Add a set up instructions describing how the reviewer should test the code
Agile board tracking
closes #515