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

Look at items and NPCs #33

Merged
merged 13 commits into from
Jan 1, 2024
Merged

Look at items and NPCs #33

merged 13 commits into from
Jan 1, 2024

Conversation

wesker-albert
Copy link
Owner

@wesker-albert wesker-albert commented Jan 1, 2024

  • add the ability to look <target> to display the description of player inventory items, items in the current room, or NPCs in the current room
  • if multiple matches exist, the search target can be prepended with a number and a period, to specify which result to look at: look 2.fork
  • the result list is sorted according to the priority the AC dictates: 1. room items, 2. inventory items, 3. NPCs
  • refactor the Look class a little bit, for better organization and readability
  • create and expose the Resources class, which will house methods the aren't necessarily associated with only one resource type

Closes #27

Copy link

github-actions bot commented Jan 1, 2024

Package Line Rate Health
. 62%
actions 100%
actions.commands 100%
actions.scheduled 100%
events 68%
models 100%
models.data 100%
resources 100%
Summary 88% (1058 / 1208)

Minimum allowed line rate is 60%

@wesker-albert wesker-albert marked this pull request as ready for review January 1, 2024 16:07
@wesker-albert wesker-albert merged commit 0610c37 into master Jan 1, 2024
1 check passed
@wesker-albert wesker-albert deleted the look-at-items-and-npcs branch January 1, 2024 16:07
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.

Look at items and NPCs
1 participant