thomaspoignant / go-feature-flag

GO Feature Flag is a simple, complete and lightweight self-hosted feature flag solution 100% Open Source. 🎛️
https://gofeatureflag.org/
MIT License
1.49k stars 148 forks source link

feat: Azure Blob Storage Exporter #2637

Closed Abeeujah closed 1 week ago

Abeeujah commented 2 weeks ago

Description

Implementation of Azure Blob Storage Exporter to export data usage of your flags to Azure Blob Storage

Closes issue(s)

Resolve #2559

Checklist

netlify[bot] commented 2 weeks ago

Deploy Preview for go-feature-flag-doc-preview ready!

Name Link
Latest commit 678342668dd7f08e0b7b8180b527a64fd3b72d2c
Latest deploy log https://app.netlify.com/sites/go-feature-flag-doc-preview/deploys/6730f2d2672e410008d3f2d4
Deploy Preview https://deploy-preview-2637--go-feature-flag-doc-preview.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

codecov[bot] commented 2 weeks ago

Codecov Report

Attention: Patch coverage is 75.00000% with 21 lines in your changes missing coverage. Please review.

Project coverage is 84.91%. Comparing base (67e6395) to head (6783426). Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
exporter/azureexporter/exporter.go 68.18% 15 Missing and 6 partials :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #2637 +/- ## ========================================== - Coverage 85.09% 84.91% -0.18% ========================================== Files 104 105 +1 Lines 4857 4940 +83 ========================================== + Hits 4133 4195 +62 - Misses 573 588 +15 - Partials 151 157 +6 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Abeeujah commented 1 week ago

Hi @thomaspoignant everything going great? Noticed you've been working on this branch, is there anything you'd have me do instead?

thomaspoignant commented 1 week ago

@Abeeujah sorry fir that, I had some issues with the tests and I've refactored them to work with azurite container. Except that it looks really great 👍

sonarcloud[bot] commented 1 week ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

Abeeujah commented 1 week ago

@Abeeujah sorry fir that, I had some issues with the tests and I've refactored them to work with azurite container. Except that it looks really great 👍

Ooh, I was just trying to be of help... Seen you merged already, GTG🚀🚀