search

fabric8-services / fabric8-auth

Identity and Access Management for fabric8 services
https://auth.openshift.io/api/status
Apache License 2.0
14 stars 26 forks source link

Remove delay between notifications #844

Closed alexeykazakov closed 5 years ago

alexeykazakov commented 5 years ago

We don't really need this delay anymore.

alien-ike commented 5 years ago

Ike Plugins (test-keeper)

Thank you @alexeykazakov for this contribution!

It appears that no tests have been added or updated in this PR.

Automated tests give us confidence in shipping reliable software. Please add some as part of this change.

If you are an admin or the reviewer of this PR and you are sure that no test is needed then you can use the command /ok-without-tests as a comment to make the status green.

Your plugin configuration is stored in the file.

codecov[bot] commented 5 years ago

Codecov Report

Merging #844 into master will decrease coverage by 0.01%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #844      +/-   ##
==========================================
- Coverage   78.09%   78.08%   -0.02%     
==========================================
  Files          99       99              
  Lines        9318     9317       -1     
==========================================
- Hits         7277     7275       -2     
- Misses       1502     1503       +1     
  Partials      539      539
Impacted Files Coverage Δ
authentication/account/service/user.go 78.01% <ø> (-0.12%) :arrow_down:
configuration/configuration.go 82.15% <ø> (-0.13%) :arrow_down:
...rovider/service/authentication_provider_service.go 72.22% <0%> (-1.12%) :arrow_down:

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 47b26c8...b70b81b. Read the comment docs.

xcoulon commented 5 years ago

/ok-without-tests