Skip to content

Elders: Handle empty quest directory gracefully#2199

Merged
manuq merged 1 commit into
mainfrom
wjt/elders-handle-empty-quest-directory-gracefully
May 15, 2026
Merged

Elders: Handle empty quest directory gracefully#2199
manuq merged 1 commit into
mainfrom
wjt/elders-handle-empty-quest-directory-gracefully

Conversation

@wjt
Copy link
Copy Markdown
Member

@wjt wjt commented May 15, 2026

Currently all three elders always have quests, but I am experimenting
with stripping out parts of the game to make the project smaller for a
game jam.

Make the elders check whether they actually have any quests for the
player, and say so if they don't.

Currently all three elders always have quests, but I am experimenting
with stripping out parts of the game to make the project smaller for a
game jam.

Make the elders check whether they actually have any quests for the
player, and say so if they don't.
@wjt wjt requested a review from a team as a code owner May 15, 2026 10:23
@wjt wjt changed the title Elders: Handle empty quest directory gracefully [1/1] Elders: Handle empty quest directory gracefully May 15, 2026
@wjt wjt changed the title [1/1] Elders: Handle empty quest directory gracefully Elders: Handle empty quest directory gracefully May 15, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 15, 2026

Test build no longer available.

LoreQuest Elder: Go forth: our world still needs your help.
=> END
~ no_quests
LoreQuest Elder: I don't have any quests for you. Talk to my fellow Elders.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@manuq manuq merged commit 226ff24 into main May 15, 2026
6 checks passed
@manuq manuq deleted the wjt/elders-handle-empty-quest-directory-gracefully branch May 15, 2026 12:47
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.

2 participants