Skip to content

Conversation

cjedro
Copy link
Collaborator

@cjedro cjedro commented Feb 20, 2025

No description provided.

Signed-off-by: Chris Jedro <cj@cloudeteer.de>
@cjedro cjedro added the ✨ enhancement New feature or request label Feb 20, 2025
Signed-off-by: Chris Jedro <cj@cloudeteer.de>
Signed-off-by: Chris Jedro <cj@cloudeteer.de>
Copy link

codecov bot commented Feb 20, 2025

Codecov Report

Attention: Patch coverage is 62.82051% with 29 lines in your changes missing coverage. Please review.

Project coverage is 32.39%. Comparing base (b7ca4b1) to head (9c547e6).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
pkg/collectors/entraid/entraid.go 67.12% 18 Missing and 6 partials ⚠️
cmd/m365-exporter/main.go 0.00% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #19      +/-   ##
==========================================
+ Coverage   30.80%   32.39%   +1.59%     
==========================================
  Files          15       16       +1     
  Lines        1490     1568      +78     
==========================================
+ Hits          459      508      +49     
- Misses        984     1007      +23     
- Partials       47       53       +6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Signed-off-by: Chris Jedro <cj@cloudeteer.de>
Signed-off-by: Chris Jedro <cj@cloudeteer.de>
Signed-off-by: Chris Jedro <cj@cloudeteer.de>
Signed-off-by: Chris Jedro <cj@cloudeteer.de>
Signed-off-by: Chris Jedro <cj@cloudeteer.de>
@cjedro cjedro marked this pull request as ready for review February 24, 2025 10:53
@cjedro cjedro enabled auto-merge (squash) February 24, 2025 10:53
@cjedro cjedro disabled auto-merge February 24, 2025 11:03
@cjedro cjedro enabled auto-merge (squash) February 24, 2025 11:04
@cjedro cjedro merged commit 0e5debf into main Feb 24, 2025
5 checks passed
@cjedro cjedro deleted the add-user-metrics branch February 24, 2025 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants