Skip to content

implement ISpeed interface#1098

Draft
microlith57 wants to merge 2 commits into
EverestAPI:devfrom
microlith57:ispeed
Draft

implement ISpeed interface#1098
microlith57 wants to merge 2 commits into
EverestAPI:devfrom
microlith57:ispeed

Conversation

@microlith57

Copy link
Copy Markdown
Member

No description provided.

@maddie480-bot maddie480-bot added the 0: draft This PR is not ready for review yet (bot-managed) label Mar 31, 2026
@swoolcock

swoolcock commented Mar 31, 2026

Copy link
Copy Markdown
Contributor

As I mentioned in Discord, I would recommend calling this something like IHasSpeed.

INoun implies that an implementation has a sole purpose (example, IEnumerator, IList). IAdjective implies that the class can perform a certain functionality but it's not its only purpose (example, IDisposable, IEnumerable). IHasNoun implies that the class provides data in addition to anything else it does (IHasXmlNode).

Edit:
Additionally, I would recommend NOT adding it to Holdable, since it's a component with convenience methods and doesn't actually use the property itself.

@balt-dev

balt-dev commented Apr 6, 2026

Copy link
Copy Markdown

don't forget PlayerSeeker!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0: draft This PR is not ready for review yet (bot-managed)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants