Closed amir4895 closed 1 year ago
@Nerzal
I think i solved the linter issue can you please re-run and add your CR comments I'll appreciate your cooperation since i need the content of the PR...
i think it'll be great if you will add " -d" flag to the linter so people will be able to see the linter problem , tnx
@Nerzal I just added the fix in the login scope for the unittests please rerun
You might also need to merge the newest commit from main, as a fix for a problem in the pipeline was merged
just did , please allow flow once more, tnx
Merging #388 (b901b08) into main (c59ba6e) will decrease coverage by
0.01%
. The diff coverage is75.00%
.
@@ Coverage Diff @@
## main #388 +/- ##
==========================================
- Coverage 77.14% 77.13% -0.02%
==========================================
Files 4 4
Lines 2140 2152 +12
==========================================
+ Hits 1651 1660 +9
- Misses 333 335 +2
- Partials 156 157 +1
Impacted Files | Coverage Δ | |
---|---|---|
models.go | 92.71% <0.00%> (-0.62%) |
:arrow_down: |
client.go | 75.46% <81.81%> (+0.03%) |
:arrow_up: |
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
Brute force protection status per user
https://github.com/Nerzal/gocloak/issues/384