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

Rework coords from separate x/y to single structs #94

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

cvet
Copy link
Owner

@cvet cvet commented Feb 4, 2024

No description provided.

…rds-rework

# Conflicts:
#	BuildTools/codegen.py
#	Source/Client/Client.cpp
#	Source/Client/CritterHexView.cpp
#	Source/Client/CritterHexView.h
#	Source/Client/ItemHexView.cpp
#	Source/Client/MapSprite.cpp
#	Source/Client/MapSprite.h
#	Source/Client/MapView.cpp
#	Source/Client/MapView.h
#	Source/Client/SpriteManager.cpp
#	Source/Client/SpriteManager.h
#	Source/Common/EngineBase.h
#	Source/Common/GenericUtils.cpp
#	Source/Common/GenericUtils.h
#	Source/Common/GeometryHelper.cpp
#	Source/Common/LineTracer.cpp
#	Source/Common/NetProtocol-Include.h
#	Source/Common/Properties.cpp
#	Source/Common/Properties.h
#	Source/Common/PropertiesSerializator.cpp
#	Source/Scripting/ClientGlobalScriptMethods.cpp
#	Source/Scripting/ClientMapScriptMethods.cpp
#	Source/Scripting/ServerGlobalScriptMethods.cpp
#	Source/Scripting/ServerMapScriptMethods.cpp
#	Source/Server/Critter.cpp
#	Source/Server/Critter.h
#	Source/Server/CritterManager.cpp
#	Source/Server/ItemManager.cpp
#	Source/Server/ItemManager.h
#	Source/Server/Map.cpp
#	Source/Server/Map.h
#	Source/Server/MapManager.cpp
#	Source/Server/Player.cpp
#	Source/Server/Player.h
#	Source/Server/Server.cpp
#	Source/Tools/Mapper.cpp
Copy link

codecov bot commented Feb 6, 2024

Codecov Report

Attention: Patch coverage is 0.07596% with 2631 lines in your changes are missing coverage. Please review.

Project coverage is 0.69%. Comparing base (ccf9109) to head (eeb8784).
Report is 7 commits behind head on master.

❗ Current head eeb8784 differs from pull request most recent head 9a81393. Consider uploading reports for the commit 9a81393 to get more accurate results

Files Patch % Lines
Source/Client/SpriteManager.cpp 0.00% 309 Missing ⚠️
Source/Tools/Mapper.cpp 0.00% 236 Missing ⚠️
Source/Server/MapManager.cpp 0.00% 214 Missing ⚠️
Source/Scripting/ServerMapScriptMethods.cpp 0.00% 156 Missing ⚠️
Source/Scripting/ClientMapScriptMethods.cpp 0.00% 143 Missing ⚠️
Source/Common/PropertiesSerializator.cpp 0.00% 134 Missing ⚠️
Source/Server/Map.cpp 0.00% 131 Missing ⚠️
Source/Client/Client.cpp 0.00% 127 Missing ⚠️
Source/Server/Server.cpp 0.00% 108 Missing ⚠️
Source/Scripting/ServerGlobalScriptMethods.cpp 0.00% 101 Missing ⚠️
... and 50 more
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #94      +/-   ##
==========================================
- Coverage    0.69%    0.69%   -0.01%     
==========================================
  Files        1753     1753              
  Lines      378494   378161     -333     
  Branches    71200    71197       -3     
==========================================
- Hits         2631     2628       -3     
+ Misses     375571   375240     -331     
- Partials      292      293       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

…rds-rework

# Conflicts:
#	Source/Client/Client.cpp
#	Source/Client/CritterHexView.cpp
#	Source/Client/MapSprite.cpp
#	Source/Client/MapSprite.h
#	Source/Client/MapView.cpp
#	Source/Client/SpriteManager.cpp
#	Source/Common/Common.h
#	Source/Common/EngineBase.h
#	Source/Common/EntityProperties.h
#	Source/Common/Properties.cpp
#	Source/Common/PropertiesSerializator.cpp
#	Source/Frontend/Application.cpp
#	Source/Frontend/Application.h
#	Source/Frontend/ApplicationHeadless.cpp
#	Source/Scripting/ServerCritterScriptMethods.cpp
#	Source/Scripting/ServerGlobalScriptMethods.cpp
#	Source/Server/CritterManager.cpp
#	Source/Server/EntityManager.cpp
#	Source/Server/MapManager.cpp
#	Source/Server/MapManager.h
#	Source/Server/Player.cpp
#	Source/Server/Server.cpp
#	Source/Server/Server.h
@therestartprince
Copy link

No complaints. Keep up the good work. fight the fight. @cvet

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants