-
Notifications
You must be signed in to change notification settings - Fork 78
feat(FR-1262): intergrated graphql for reservoir #4157
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
Conversation
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has required the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
Coverage report for
|
St.❔ |
Category | Percentage | Covered / Total |
|---|---|---|---|
| 🔴 | Statements | 4.48% (+0.03% 🔼) |
501/11174 |
| 🔴 | Branches | 3.63% (+0.03% 🔼) |
285/7858 |
| 🔴 | Functions | 2.59% (+0.01% 🔼) |
90/3476 |
| 🔴 | Lines | 4.46% (+0.03% 🔼) |
487/10925 |
Show new covered files 🐣
St.❔ |
File | Statements | Branches | Functions | Lines |
|---|---|---|---|---|---|
| 🔴 | ... / ReservoirArtifactDetailPage.tsx |
0% | 0% | 0% | 0% |
Test suite run success
114 tests passing in 13 suites.
Report generated by 🧪jest coverage report action from 74b18be
d715fd0 to
91a0d82
Compare
d07e960 to
1409210
Compare
Coverage report for
|
St.❔ |
Category | Percentage | Covered / Total |
|---|---|---|---|
| 🔴 | Statements | 52.42% | 130/248 |
| 🔴 | Branches | 32.26% | 80/248 |
| 🔴 | Functions | 35.09% | 20/57 |
| 🔴 | Lines | 54.84% | 119/217 |
Test suite run success
55 tests passing in 3 suites.
Report generated by 🧪jest coverage report action from 74b18be
91a0d82 to
aea9d8f
Compare
188a7db to
a64ebbd
Compare
aea9d8f to
e29c8a7
Compare
f0fcf90 to
81d7fd4
Compare
e29c8a7 to
3093243
Compare
3093243 to
3905acd
Compare
81d7fd4 to
72baf49
Compare
c5dd75a to
bdab7c3
Compare
4ebb2ac to
75c0949
Compare
bdab7c3 to
c55f576
Compare
75c0949 to
d45d03d
Compare
ad057bf to
e59b40a
Compare
08b00d9 to
c43778b
Compare
e59b40a to
096d50d
Compare
c43778b to
c076370
Compare
096d50d to
5cabe48
Compare
5cabe48 to
eb503f9
Compare
eb503f9 to
e86d92d
Compare
c076370 to
6561408
Compare
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
Merge activity
|
resolves #3980 (FR-1262) <!-- replace NNN, MMM with the GitHub issue number and the corresponding Jira issue number. --> <!-- Please precisely, concisely, and concretely describe what this PR changes, the rationale behind codes, and how it affects the users and other developers. --> **Checklist:** (if applicable) - [ ] Documentation - [ ] Minium required manager version - [ ] Specific setting for review (eg., KB link, endpoint or how to setup) - [ ] Minimum requirements to check during review - [ ] Test case(s) to demonstrate the difference of before/after
6561408 to
962fcde
Compare
e86d92d to
74b18be
Compare

resolves #3980 (FR-1262)
Checklist: (if applicable)