8000 MF-1378 - Update dependencies by dborovcanin · Pull Request #1379 · absmach/supermq · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

MF-1378 - Update dependencies #1379

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

Merged
merged 6 commits into from
May 20, 2021
Merged

Conversation

dborovcanin
Copy link
Collaborator

Signed-off-by: dusanb94 dusan.borovcanin@mainflux.com

What does this do?

This pull request updates Mainflux services dependencies.

Which issue(s) does this PR fix/relate to?

This pull request resolves #1378.

List any changes that modify/break current functionality

There are no such changes.

Have you included tests for your changes?

No, but I did manual testing.

Did you document any new/modified functionality?

N/A.

@codecov-io
Copy link

Codecov Report

Merging #1379 (d36c0a8) into master (530f925) will increase coverage by 0.02%.
The diff coverage is 72.05%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1379      +/-   ##
==========================================
+ Coverage   67.51%   67.54%   +0.02%     
==========================================
  Files         121      121              
  Lines        8408     8408              
==========================================
+ Hits         5677     5679       +2     
+ Misses       2216     2214       -2     
  Partials      515      515              
Impacted Files Coverage Δ
bootstrap/service.go 74.34% <ø> (ø)
things/redis/channels.go 64.70% <25.00%> (ø)
twins/redis/twins.go 41.46% <38.46%> (ø)
things/redis/things.go 66.66% <40.00%> (ø)
bootstrap/redis/producer/streams.go 92.95% <75.00%> (ø)
bootstrap/api/endpoint.go 100.00% <100.00%> (ø)
things/redis/streams.go 92.66% <100.00%> (ø)
things/service.go 69.50% <0.00%> (+1.41%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 530f925...d36c0a8. Read the comment docs.

@dborovcanin dborovcanin force-pushed the update-dependencies branch from b8ec32c to 5867c40 Compare March 5, 2021 12:03
@dborovcanin dborovcanin force-pushed the update-dependencies branch 2 times, most recently from 27b4230 to ab3d722 Compare May 13, 2021 13:05
@codecov-commenter
Copy link
codecov-commenter commented May 13, 2021

Codecov Report

Merging #1379 (762fc30) into master (38ca7f7) will not change coverage.
The diff coverage is 66.26%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1379   +/-   ##
=======================================
  Coverage   69.26%   69.26%           
=======================================
  Files         123      123           
  Lines        8452     8452           
=======================================
  Hits         5854     5854           
  Misses       2072     2072           
  Partials      526      526           
Impacted Files Coverage Δ
bootstrap/service.go 74.34% <ø> (ø)
things/redis/channels.go 64.70% <25.00%> (ø)
twins/redis/twins.go 41.46% <38.46%> (ø)
lora/adapter.go 62.00% <40.00%> (ø)
things/redis/things.go 66.66% <40.00%> (ø)
bootstrap/redis/producer/streams.go 92.95% <75.00%> (ø)
bootstrap/api/endpoint.go 100.00% <100.00%> (ø)
things/redis/streams.go 92.66% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 38ca7f7...762fc30. Read the comment docs.

@dborovcanin dborovcanin force-pushed the update-dependencies branch 2 times, most recently from 3e65745 to 02085f9 Compare May 14, 2021 09:50
Signed-off-by: dusanb94 <dusan.borovcanin@mainflux.com>
Signed-off-by: dusanb94 <dusan.borovcanin@mainflux.com>
@dborovcanin dborovcanin marked this pull request as ready for review May 14, 2021 14:53
@dborovcanin dborovcanin requested a review from a team as a code owner May 14, 2021 14:53
@dborovcanin dborovcanin force-pushed the update-dependencies branch from 2c747e4 to 762fc30 Compare May 17, 2021 08:50
manuio
manuio previously approved these changes May 18, 2021
Copy link
Contributor
@manuio manuio left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: dusanb94 <dusan.borovcanin@mainflux.com>
Signed-off-by: dusanb94 <dusan.borovcanin@mainflux.com>
Signed-off-by: dusanb94 <dusan.borovcanin@mainflux.com>
Signed-off-by: dusanb94 <dusan.borovcanin@mainflux.com>
Copy link
Contributor
@manuio manuio left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor
@drasko drasko left a comment

Choose a reason for hiding this comment

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

LGTM

@drasko drasko merged commit 516c02b into absmach:master May 20, 2021
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.

Update dependencies
5 participants
0