Skip to content

the more you rewrite them the fresher they get#1072

Merged
drazisil merged 28 commits intomainfrom
fresh-tests
Jan 29, 2022
Merged

the more you rewrite them the fresher they get#1072
drazisil merged 28 commits intomainfrom
fresh-tests

Conversation

@drazisil
Copy link
Collaborator

No description provided.

@codecov
Copy link

codecov bot commented Jan 22, 2022

Codecov Report

Merging #1072 (bb02296) into main (59d2db0) will decrease coverage by 27.90%.
The diff coverage is 16.04%.

Impacted Files Coverage Δ
src/core/listener-thread.ts 0.00% <0.00%> (ø)
src/core/tcpConnection.ts 6.54% <0.00%> (-35.02%) ⬇️
src/message-types/index.ts 10.78% <ø> (-53.94%) ⬇️
src/server/sslListener.ts 0.00% <0.00%> (ø)
src/shard/index.ts 0.00% <ø> (-29.12%) ⬇️
src/transactions/transactionServer.ts 2.52% <2.52%> (ø)
src/core/connection-mgr.ts 17.29% <14.63%> (-1.30%) ⬇️
src/admin/adminServer.ts 28.88% <28.88%> (ø)
src/persona/index.ts 15.62% <58.82%> (-45.67%) ⬇️
src/admin/index.ts 100.00% <100.00%> (ø)
... and 22 more

@lgtm-com
Copy link

lgtm-com bot commented Jan 22, 2022

This pull request fixes 5 alerts when merging 3f1ea44 into 59d2db0 - view on LGTM.com

fixed alerts:

  • 5 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Jan 22, 2022

This pull request fixes 5 alerts when merging 3322de4 into 59d2db0 - view on LGTM.com

fixed alerts:

  • 5 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Jan 22, 2022

This pull request fixes 5 alerts when merging 1c77201 into 59d2db0 - view on LGTM.com

fixed alerts:

  • 5 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Jan 22, 2022

This pull request fixes 5 alerts when merging e92c611 into 59d2db0 - view on LGTM.com

fixed alerts:

  • 5 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Jan 22, 2022

This pull request introduces 1 alert and fixes 5 when merging f589c54 into 59d2db0 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

fixed alerts:

  • 5 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Jan 23, 2022

This pull request fixes 5 alerts when merging 1b98fc8 into 59d2db0 - view on LGTM.com

fixed alerts:

  • 5 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Jan 23, 2022

This pull request introduces 1 alert and fixes 5 when merging c1580c6 into 59d2db0 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

fixed alerts:

  • 5 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Jan 23, 2022

This pull request introduces 2 alerts and fixes 5 when merging 203c054 into 59d2db0 - view on LGTM.com

new alerts:

  • 2 for Unused variable, import, function or class

fixed alerts:

  • 5 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Jan 23, 2022

This pull request fixes 5 alerts when merging da9fe6f into 59d2db0 - view on LGTM.com

fixed alerts:

  • 5 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Jan 24, 2022

This pull request fixes 5 alerts when merging 6235293 into 59d2db0 - view on LGTM.com

fixed alerts:

  • 5 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Jan 24, 2022

This pull request fixes 5 alerts when merging 25c9789 into 59d2db0 - view on LGTM.com

fixed alerts:

  • 5 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Jan 24, 2022

This pull request fixes 5 alerts when merging 14b36ab into 59d2db0 - view on LGTM.com

fixed alerts:

  • 5 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Jan 24, 2022

This pull request fixes 5 alerts when merging 559353b into 59d2db0 - view on LGTM.com

fixed alerts:

  • 5 for Hard-coded credentials

@lgtm-com
Copy link

lgtm-com bot commented Jan 26, 2022

This pull request fixes 5 alerts when merging bb02296 into 59d2db0 - view on LGTM.com

fixed alerts:

  • 5 for Hard-coded credentials

@drazisil drazisil merged commit c953180 into main Jan 29, 2022
@drazisil drazisil deleted the fresh-tests branch January 29, 2022 23:34
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.

2 participants