Skip to content

Dev#10

Merged
JunkyDeveloper merged 21 commits into
FlintTestMC:mainfrom
JunkyDeveloper:dev
Feb 5, 2026
Merged

Dev#10
JunkyDeveloper merged 21 commits into
FlintTestMC:mainfrom
JunkyDeveloper:dev

Conversation

@JunkyDeveloper

@JunkyDeveloper JunkyDeveloper commented Feb 1, 2026

Copy link
Copy Markdown
Contributor

This includes the #9 PR and flint-steel which is more an abstracted flint-lib version

@JunkyDeveloper JunkyDeveloper mentioned this pull request Feb 1, 2026

@coco875 coco875 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

mostly look good, just avoid to duplicate struct

Comment thread src/traits.rs Outdated

/// An item that can be held or placed in a slot
#[derive(Debug, Clone, PartialEq, Eq)]
pub struct Item {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

already define in test_specs

Comment thread src/traits.rs Outdated

/// Block data returned from get_block
#[derive(Debug, Clone, PartialEq, Eq, Serialize)]
pub struct BlockData {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

already define in test_specs

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

also add clippy

@JunkyDeveloper
JunkyDeveloper merged commit 2222055 into FlintTestMC:main Feb 5, 2026
3 checks passed
@JunkyDeveloper
JunkyDeveloper deleted the dev branch February 5, 2026 21:10
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