Skip to content

ci: fix gametest structure missing#113

Merged
lonelyicer merged 1 commit intomainfrom
ci/fix-gametest-failed
Mar 16, 2026
Merged

ci: fix gametest structure missing#113
lonelyicer merged 1 commit intomainfrom
ci/fix-gametest-failed

Conversation

@lonelyicer
Copy link
Member

No description provided.

Copilot AI review requested due to automatic review settings March 16, 2026 20:21
@github-actions github-actions bot added the ci ci label Mar 16, 2026
@lonelyicer lonelyicer enabled auto-merge March 16, 2026 20:22
@lonelyicer lonelyicer merged commit ed37319 into main Mar 16, 2026
5 checks passed
@lonelyicer lonelyicer deleted the ci/fix-gametest-failed branch March 16, 2026 20:23
@github-actions
Copy link

Game Test Results

✅ Success

Metric Value
Total Tests 1
Passed 1
Failed 0
Duration 358.6 ms

Workflow run: #393

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR appears intended to fix CI failures caused by a missing GameTest structure template by adding the referenced NBT template to resources and updating the test to use it.

Changes:

  • Added a new NBT structure template at data/mia/structure/gametest/amethyst_lamptube.nbt.
  • Updated MiaGameTests.exampleTest to reference the new template path (gametest/amethyst_lamptube) instead of test_template.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
src/main/resources/data/mia/structure/gametest/amethyst_lamptube.nbt Adds the GameTest template NBT resource intended to satisfy template loading.
src/main/java/com/altnoir/mia/MiaGameTests.java Updates the @GameTest(template=...) value to point at the new template.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

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

Labels

ci ci

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants