Skip to content
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

Libms remove gitlab mode #435

Conversation

prasadtalasila
Copy link
Contributor

The gitlab mode of library microservice is broken.
It is removed from the codebase to make
library microservice functional again.

This PR partially addresses the problems mentioned in issues #167 and #218.
This PR also includes the code refactoring suggested in PR #326. Thanks @OliverGeneser for the work.

@prasadtalasila prasadtalasila added this to the Release v0.4.0 milestone Jan 9, 2024
@codecov-commenter
Copy link

codecov-commenter commented Jan 9, 2024

Codecov Report

Merging #435 (06239d1) into feature/distributed-demo (abc90a6) will increase coverage by 15.71%.
The diff coverage is 100.00%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@                      Coverage Diff                      @@
##           feature/distributed-demo     #435       +/-   ##
=============================================================
+ Coverage                     66.36%   82.07%   +15.71%     
=============================================================
  Files                            35        6       -29     
  Lines                           440      106      -334     
  Branches                         29        4       -25     
=============================================================
- Hits                            292       87      -205     
+ Misses                          130       15      -115     
+ Partials                         18        4       -14     
Files Coverage Δ
servers/lib/src/files/files.module.ts 100.00% <ø> (ø)
...rs/lib/src/files/services/files-service.factory.ts 90.00% <ø> (ø)
...vers/lib/src/files/services/local-files.service.ts 88.88% <100.00%> (ø)

... and 38 files with indirect coverage changes

Components Coverage Δ
Website ∅ <ø> (∅)
Lib Microservice 82.07% <100.00%> (∅)

@prasadtalasila prasadtalasila force-pushed the libms-remove-gitlab-mode branch from 2bc08a6 to 4d70329 Compare January 12, 2024 11:12
@@ -159,7 +159,8 @@

??? Question "Does DTaaS support data collection from different sources like hardware, software and network? Is there any user interface or any tracking instruments used for data collection?"

The DTaaS provids InfluxDB, RabbitMQ, MQTT services. Both the physical twin
The DTaaS provids InfluxDB, RabbitMQ, MQTT and MongoDB services.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code block style

Copy link

codeclimate bot commented Jan 16, 2024

Code Climate has analyzed commit 06239d1 and detected 1 issue on this pull request.

Here's the issue category breakdown:

Category Count
Style 1

View more on Code Climate.

@prasadtalasila prasadtalasila merged commit 105044c into INTO-CPS-Association:feature/distributed-demo Jan 16, 2024
3 checks passed
@prasadtalasila prasadtalasila deleted the libms-remove-gitlab-mode branch January 17, 2024 08:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants