EXUI-4298: Suppressions - express - major#2163
Merged
Merged
Conversation
connorpgpmcelroy
previously approved these changes
May 18, 2026
connorpgpmcelroy
left a comment
connorpgpmcelroy
left a comment
Contributor
There was a problem hiding this comment.
If this still works fine, then happy to approve, especially if getting rid of dead code. However we definitely need to be sure styling isn't affected subsequently in the consuming applications
RiteshHMCTS
previously approved these changes
May 18, 2026
chrisjones-hmcts
dismissed stale reviews from RiteshHMCTS and connorpgpmcelroy
via
72a504f
|
RiteshHMCTS
approved these changes
Jun 11, 2026
Josh-HMCTS
approved these changes
Jun 11, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Jira link
See EXUI-4298
Change description
Removed express and storybook dependencies
Testing done
Compared the published package with these changes against the previous package, using the EXUI-4425 publishing changes. The packages were identical except for the version field in package.json.
Dev tested and all other tests pass
MC PR: hmcts/rpx-xui-webapp#5140
CVE Suppression: Are there any CVEs present in the codebase (either newly introduced or pre-existing) that are being intentionally suppressed or ignored by this commit?