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

Exchange with relation targets, more batch ops #342

Merged
merged 12 commits into from
Jan 18, 2024
Merged

Conversation

mlange-42
Copy link
Owner

@mlange-42 mlange-42 commented Jan 18, 2024

Adds methods to allow for exchanging components and setting relation targets at the same time. Includes batch operations and generic API.

  • adds methods Relations.Exchange(), Relations.ExchangeBatch(), Relations.ExchangeBatchQ() for exchange with relation target
  • adds Exchange.WithRelation() and optional target argument for operations with relation target
  • adds MapX.AddBatch(), MapX.AddBatchQ(), MapX.RemoveBatch()and MapX.RemoveBatchQ()
  • adds optional relation target argument to most MapX methods
  • adds FilterX.Filter() to get an ecs.Filter from a generic one

@mlange-42 mlange-42 added usability Usability and ergonomics feature feature requests labels Jan 18, 2024
@mlange-42 mlange-42 self-assigned this Jan 18, 2024
@coveralls
Copy link
Collaborator

coveralls commented Jan 18, 2024

Pull Request Test Coverage Report for Build 7576871254

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 7573644274: 0.0%
Covered Lines: 6036
Relevant Lines: 6036

💛 - Coveralls

@mlange-42 mlange-42 changed the title Exchange with relation targets Exchange with relation targets, more batch ops Jan 18, 2024
@mlange-42 mlange-42 force-pushed the exchange-relation branch 2 times, most recently from 7b92f6a to 9d5d034 Compare January 18, 2024 22:36
@mlange-42 mlange-42 merged commit a22df17 into main Jan 18, 2024
18 checks passed
@mlange-42 mlange-42 deleted the exchange-relation branch January 18, 2024 23:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature feature requests usability Usability and ergonomics
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants