-
Notifications
You must be signed in to change notification settings - Fork 3
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
Module Federation #619
Module Federation #619
Conversation
Hello monpote,My role is to assist you with the merge of this Status report is not available. |
Codecov Report
@@ Coverage Diff @@
## development/2.1 #619 +/- ##
===================================================
+ Coverage 57.23% 58.14% +0.91%
===================================================
Files 227 208 -19
Lines 10793 10152 -641
Branches 3061 2940 -121
===================================================
- Hits 6177 5903 -274
+ Misses 4600 4233 -367
Partials 16 16
|
c8093c5
to
8b7e224
Compare
7c5946c
to
eb85fbc
Compare
8b53adb
to
f98c20d
Compare
baa19aa
to
80a2b15
Compare
bceb8fa
to
7eb1e2d
Compare
f6bc6f7
to
ee9db56
Compare
ping |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list: |
ee9db56
to
6a4efab
Compare
ZKUI-371 - update webpack configs ZKUI-371 - update webpack's deps ZKUI-371 - fix build fix svg warning ZKUI-371 - fix security check fix dev webpack environnement fix monaco issue replace push from connected-react-router for history in react component remove navbar fix user creation fix grafana url fix all tests update core-ui fix rebase Remove legacy cypress tests fix linter errors make prod ready change public path fix sts endpoint fix debug fix change client endpoint Solve s3 proxy issues for object with keys containing urlencoded chars This commit is the integration of #554 fix s3client add basePath in s3client Fix security workflow fix test Add go template fix ci Remove index.html remove default to index.html fix basepath for zenko client
6a4efab
to
25963f3
Compare
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list: |
/approve |
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 !
/status |
StatusStatus report is not available. The following options are set: approve |
dcf3b78
to
e69acfd
Compare
e69acfd
to
01490d0
Compare
In the queueThe changeset has received all authorizations and has been added to the The changeset will be merged in:
The following branches will NOT be impacted:
There is no action required on your side. You will be notified here once IMPORTANT Please do not attempt to modify this pull request.
If you need this pull request to be removed from the queue, please contact a The following options are set: approve |
I have successfully merged the changeset of this pull request
The following branches have NOT changed:
Please check the status of the associated issue ZKUI-371. Goodbye monpote. |
No description provided.