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

fix ownership status assignment in getDevices() #256

Merged
merged 1 commit into from
May 26, 2022
Merged

Conversation

der-dano
Copy link
Collaborator

No description provided.

@der-dano der-dano requested a review from jkralik May 26, 2022 13:09
@sonarqubecloud
Copy link

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

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@codecov-commenter
Copy link

Codecov Report

Merging #256 (9ed6569) into main (251c41e) will increase coverage by 0.01%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##             main     #256      +/-   ##
==========================================
+ Coverage   60.41%   60.43%   +0.01%     
==========================================
  Files          63       63              
  Lines        4007     4006       -1     
==========================================
  Hits         2421     2421              
- Misses       1246     1247       +1     
+ Partials      340      338       -2     
Impacted Files Coverage Δ
client/getDevices.go 62.87% <50.00%> (+0.37%) ⬆️
client/core/disownDevice.go 72.72% <0.00%> (-3.04%) ⬇️
client/core/maintenance.go 62.96% <0.00%> (+3.70%) ⬆️

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 251c41e...9ed6569. Read the comment docs.

@jkralik jkralik merged commit 4a71b9e into main May 26, 2022
@jkralik jkralik deleted the fix-ownership-status branch May 26, 2022 13:14
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.

3 participants