Skip to content

Rename Route:query() #19

@YetAnotherClown

Description

@YetAnotherClown

Thanks to Ukendio for the input regarding this issue!

The Route:query() method is misleading, the query keyword should be reserved for the ECS that YetAnotherNet is integrating with.

Suggested names for the method are:

  • Route:drain()
  • Route:read()

Other names may be suggested in the comments here.

The QueryResult object should also be renamed to reflect this. These changes are also reserved for version 1.0.0 to avoid introducing breaking changes for issues like this, and instead fix them all in the official 1.0.0 release.

Metadata

Metadata

Assignees

No one assigned

    Labels

    reserved for version 1.0.0This issue will not be resolved until development of version 1.0.0

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions