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

refactor(cardinal): register query with generics #413

Merged
merged 4 commits into from
Nov 13, 2023

Conversation

smsunarto
Copy link
Member

@smsunarto smsunarto commented Nov 9, 2023

Closes: #XXX

What is the purpose of the change

Making register query more consistent with register component

Brief Changelog

  • Refactor RegisterQuery to be similar to RegisterComponent

Testing and Verifying

  • All relevant tests have been adjusted accordingly

Copy link
Member Author

smsunarto commented Nov 9, 2023

Copy link

codecov bot commented Nov 9, 2023

Codecov Report

Merging #413 (622b1dd) into main (f091e73) will decrease coverage by 0.07%.
The diff coverage is 75.69%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #413      +/-   ##
==========================================
- Coverage   64.78%   64.71%   -0.07%     
==========================================
  Files          82       81       -1     
  Lines        4682     4713      +31     
==========================================
+ Hits         3033     3050      +17     
- Misses       1315     1329      +14     
  Partials      334      334              
Files Coverage Δ
cardinal/message.go 92.50% <100.00%> (ø)
cardinal/search.go 100.00% <100.00%> (ø)
cardinal/testing.go 55.55% <100.00%> (ø)
cardinal/util.go 83.33% <ø> (-4.17%) ⬇️
cardinal/world_context.go 78.57% <100.00%> (ø)
cardinal/ecs/query.go 56.20% <97.29%> (+4.41%) ⬆️
cardinal/ecs/world.go 66.18% <80.95%> (+0.42%) ⬆️
cardinal/evm/server.go 41.20% <64.28%> (+0.48%) ⬆️
cardinal/world.go 67.85% <68.75%> (-1.47%) ⬇️
cardinal/server/query.go 53.17% <37.50%> (-4.73%) ⬇️

@smsunarto smsunarto force-pushed the 11-08-refactor_upstream_config_and_refactor_NewWorld branch from cb0f00a to 4c480dd Compare November 9, 2023 07:01
@smsunarto smsunarto force-pushed the 11-08-refactor_register_query branch from d4f19a0 to b7e044f Compare November 9, 2023 07:01
cardinal/ecs/query.go Show resolved Hide resolved
cardinal/ecs/query.go Outdated Show resolved Hide resolved
cardinal/ecs/query_test.go Outdated Show resolved Hide resolved
cardinal/ecs/query.go Outdated Show resolved Hide resolved
cardinal/ecs/query_test.go Outdated Show resolved Hide resolved
@smsunarto smsunarto force-pushed the 11-08-refactor_upstream_config_and_refactor_NewWorld branch from 4fc6230 to 8dd7401 Compare November 11, 2023 01:53
@smsunarto smsunarto force-pushed the 11-08-refactor_register_query branch 2 times, most recently from f24e962 to 237270f Compare November 11, 2023 02:08
@smsunarto smsunarto force-pushed the 11-08-refactor_upstream_config_and_refactor_NewWorld branch from 8a543c3 to ef33794 Compare November 11, 2023 05:24
@smsunarto smsunarto force-pushed the 11-08-refactor_register_query branch 4 times, most recently from 2242ded to b71e3a5 Compare November 11, 2023 06:32
cardinal/ecs/query_test.go Outdated Show resolved Hide resolved
cardinal/ecs/query_test.go Outdated Show resolved Hide resolved
@smsunarto smsunarto force-pushed the 11-08-refactor_upstream_config_and_refactor_NewWorld branch from 2a146b2 to 62cc6db Compare November 13, 2023 20:12
@smsunarto smsunarto force-pushed the 11-08-refactor_register_query branch 3 times, most recently from 7c3c439 to 73f0685 Compare November 13, 2023 20:24
@smsunarto smsunarto force-pushed the 11-08-refactor_upstream_config_and_refactor_NewWorld branch 2 times, most recently from c18cd7f to 991914e Compare November 13, 2023 20:27
@smsunarto smsunarto force-pushed the 11-08-refactor_register_query branch 2 times, most recently from 860386e to 786b65d Compare November 13, 2023 20:38
Base automatically changed from 11-08-refactor_upstream_config_and_refactor_NewWorld to main November 13, 2023 20:48
Copy link
Member Author

smsunarto commented Nov 13, 2023

Merge activity

  • Nov 13, 3:48 PM: Graphite rebased this pull request after merging its parent, because this pull request is set to merge when ready.
  • Nov 13, 4:31 PM: @smsunarto merged this pull request with Graphite.

@smsunarto smsunarto enabled auto-merge (squash) November 13, 2023 20:50
@smsunarto
Copy link
Member Author

gdi, stop merging in front of me

@smsunarto
Copy link
Member Author

fuck e2e tests, all my homies hate the e2e

@smsunarto smsunarto merged commit 27f1dc7 into main Nov 13, 2023
9 checks passed
@smsunarto smsunarto deleted the 11-08-refactor_register_query branch November 13, 2023 21:31
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.

3 participants