Skip to content
This repository has been archived by the owner on Dec 21, 2023. It is now read-only.

chore(bridge): Remove unused dependencies #7012

Merged
merged 3 commits into from
Mar 10, 2022
Merged

Conversation

mowies
Copy link
Member

@mowies mowies commented Feb 28, 2022

This PR

  • removes some unused dependencies from bridge and bridge server

@codecov
Copy link

codecov bot commented Feb 28, 2022

Codecov Report

Merging #7012 (0e57a54) into master (6b3396c) will increase coverage by 0.64%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #7012      +/-   ##
==========================================
+ Coverage   57.83%   58.47%   +0.64%     
==========================================
  Files         514      523       +9     
  Lines       29967    30364     +397     
  Branches     1408     1501      +93     
==========================================
+ Hits        17332    17756     +424     
+ Misses      11383    11361      -22     
+ Partials     1252     1247       -5     
Impacted Files Coverage Δ
bridge/client/app/_utils/app.utils.ts 82.75% <0.00%> (-17.25%) ⬇️
bridge/client/app/_services/api.service.mock.ts 70.39% <0.00%> (-1.91%) ⬇️
...s/ktb-sequence-view/ktb-sequence-view.component.ts 48.14% <0.00%> (-0.78%) ⬇️
bridge/client/app/_models/project.ts 60.24% <0.00%> (-0.57%) ⬇️
bridge/client/app/app.module.ts 100.00% <0.00%> (ø)
bridge/shared/interfaces/event-types.ts 100.00% <0.00%> (ø)
...b-no-service-info/ktb-no-service-info.component.ts 100.00% <0.00%> (ø)
bridge/server/interfaces/webhook-config-yaml.ts
bridge/client/app/_models/trigger-sequence.ts 100.00% <0.00%> (ø)
...app/_directives/overlay-service/overlay.service.ts 95.45% <0.00%> (ø)
... and 15 more
Flag Coverage Δ
bridge-server 66.31% <ø> (+8.01%) ⬆️
bridge2 73.48% <ø> (+0.06%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

re-add ts-node

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>

remove some packages that seem to be unneeded/unused

Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
@sonarcloud
Copy link

sonarcloud bot commented Mar 9, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@mowies mowies marked this pull request as ready for review March 9, 2022 13:40
@mowies mowies merged commit 9be7608 into master Mar 10, 2022
@mowies mowies deleted the chore/clean-up-deps branch March 10, 2022 14:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants