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

Update Gametest mappings #324

Merged
merged 5 commits into from Jan 22, 2023
Merged

Update Gametest mappings #324

merged 5 commits into from Jan 22, 2023

Conversation

Pyrofab
Copy link
Member

@Pyrofab Pyrofab commented Dec 25, 2022

Noticed some names were a bit misleading, so I tried to clarify where I could.

Note: Mojmaps have been used to crosscheck intent a couple times.

mappings/net/minecraft/test/TestContext.mapping Outdated Show resolved Hide resolved
mappings/net/minecraft/test/GameTest.mapping Outdated Show resolved Hide resolved
mappings/net/minecraft/test/GameTest.mapping Outdated Show resolved Hide resolved
mappings/net/minecraft/test/TestContext.mapping Outdated Show resolved Hide resolved
@ix0rai ix0rai added s: small PRs with less than 200 lines t: new adds new mappings t: refactor proposes a refactor v: release targets a release version of minecraft labels Dec 26, 2022
Copy link
Member

@ix0rai ix0rai left a comment

Choose a reason for hiding this comment

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

amazing work!

@ix0rai
Copy link
Member

ix0rai commented Dec 26, 2022

build failure is my own fault -- sorry. {@link Identifier} needs to be {@link net.minecraft.util.Identifier}

Co-authored-by: ix0rai <ix0rai64@gmail.com>
@ix0rai ix0rai added the reviews needed please review this PR label Dec 26, 2022
mappings/net/minecraft/test/GameTest.mapping Outdated Show resolved Hide resolved
mappings/net/minecraft/test/GameTest.mapping Outdated Show resolved Hide resolved
mappings/net/minecraft/test/TestContext.mapping Outdated Show resolved Hide resolved
@NoComment1105 NoComment1105 added the update-base used to notify github actions that the base branch should be updated label Jan 18, 2023
@github-actions
Copy link
Contributor

🚀 Target branch has been updated to 23w03a

@github-actions github-actions bot changed the base branch from 1.19.3 to 23w03a January 18, 2023 16:42
@github-actions github-actions bot added v: snapshot targets a snapshot version of minecraft and removed update-base used to notify github actions that the base branch should be updated v: release targets a release version of minecraft labels Jan 18, 2023
@OroArmor OroArmor added final-comment-period is approved and will soon be merged if no issues are raised and removed reviews needed please review this PR labels Jan 20, 2023
@OroArmor OroArmor merged commit 0d2e292 into QuiltMC:23w03a Jan 22, 2023
OroArmor pushed a commit that referenced this pull request Jan 22, 2023
* Update Gametest mappings

* Clarify inverted assertions

* Apply suggestions from code review

Co-authored-by: ix0rai <ix0rai64@gmail.com>

* Rename GameTest#tickLimit to timeout

Co-authored-by: ix0rai <ix0rai64@gmail.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
OroArmor pushed a commit that referenced this pull request Jan 22, 2023
* Update Gametest mappings

* Clarify inverted assertions

* Apply suggestions from code review

Co-authored-by: ix0rai <ix0rai64@gmail.com>

* Rename GameTest#tickLimit to timeout

Co-authored-by: ix0rai <ix0rai64@gmail.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
final-comment-period is approved and will soon be merged if no issues are raised s: small PRs with less than 200 lines t: new adds new mappings t: refactor proposes a refactor v: snapshot targets a snapshot version of minecraft
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants