Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Twelve Factor App #781

Merged
merged 57 commits into from
Jan 22, 2021
Merged

Twelve Factor App #781

merged 57 commits into from
Jan 22, 2021

Conversation

drazisil
Copy link
Collaborator

@codecov
Copy link

codecov bot commented Dec 16, 2020

Codecov Report

Merging #781 (89a0b4d) into master (7208189) will increase coverage by 1.20%.
The diff coverage is 71.42%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #781      +/-   ##
==========================================
+ Coverage   46.65%   47.86%   +1.20%     
==========================================
  Files          33       33              
  Lines        1479     1519      +40     
  Branches       61       61              
==========================================
+ Hits          690      727      +37     
- Misses        789      792       +3     
Impacted Files Coverage Δ
src/services/MCServer/LoginServer/packet.js 100.00% <ø> (ø)
src/services/MCServer/index.js 47.05% <0.00%> (ø)
src/services/PatchAndShard/patchServer.js 91.66% <ø> (ø)
src/services/AuthLogin/AuthLogin.js 33.33% <50.00%> (+14.72%) ⬆️
src/services/AdminServer/AdminServer.js 29.68% <76.92%> (+14.87%) ⬆️
src/shared/databaseManager.js 37.93% <88.88%> (+18.88%) ⬆️
config/app-settings.js 100.00% <100.00%> (ø)
src/services/MCServer/LobbyServer/LobbyServer.js 18.09% <100.00%> (+0.78%) ⬆️
src/services/MCServer/MCOTS/TCPManager.js 8.43% <100.00%> (+0.55%) ⬆️

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 7208189...89a0b4d. Read the comment docs.

renovate-bot and others added 25 commits December 17, 2020 18:58
…-1.x

Update codecov/codecov-action action to v1.1.0
…-1.x

Update codecov/codecov-action action to v1.1.1
Update dependency snyk to v1.437.3
Update dependency @types/node to v14.14.14
Update dependency @types/node to v14.14.16
Update dependency sinon to v9.2.3
@lgtm-com
Copy link

lgtm-com bot commented Jan 9, 2021

This pull request introduces 4 alerts when merging caf03ef into 9505a95 - view on LGTM.com

new alerts:

  • 4 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Jan 9, 2021

This pull request introduces 4 alerts when merging b2c2762 into 9505a95 - view on LGTM.com

new alerts:

  • 4 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Jan 9, 2021

This pull request introduces 4 alerts when merging 179d7b5 into 9505a95 - view on LGTM.com

new alerts:

  • 4 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Jan 9, 2021

This pull request introduces 4 alerts when merging 01c5db8 into 9505a95 - view on LGTM.com

new alerts:

  • 4 for Unused variable, import, function or class

@drazisil drazisil force-pushed the twelve-factor-app branch 2 times, most recently from f543ec4 to 1c9b0eb Compare January 9, 2021 17:00
@ghost
Copy link

ghost commented Jan 22, 2021

Congratulations 🎉. DeepCode analyzed your code in 2.896 seconds and we found no issues. Enjoy a moment of no bugs ☀️.

👉 View analysis in DeepCode’s Dashboard | Configure the bot

@drazisil drazisil merged commit bc565ba into master Jan 22, 2021
@drazisil drazisil deleted the twelve-factor-app branch January 22, 2021 01:37
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.

None yet

2 participants