Skip to content

CCSMB-15: Wide Printer Format (2dj) and Wide Printer Format Array (2dja) Formats and CCSMB-16: 3D Print JSON (3dj)#31

Open
znepb wants to merge 6 commits intomasterfrom
2dj-2dja-3dj
Open

CCSMB-15: Wide Printer Format (2dj) and Wide Printer Format Array (2dja) Formats and CCSMB-16: 3D Print JSON (3dj)#31
znepb wants to merge 6 commits intomasterfrom
2dj-2dja-3dj

Conversation

@znepb
Copy link

@znepb znepb commented Mar 30, 2024

RFCs for 2dj, 2dja, and 3dj files. Suggestions welcome!

Copy link
Member

@EmmaKnijn EmmaKnijn left a comment

Choose a reason for hiding this comment

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

Looks good, just change the titles and you'll be good

@EmmaKnijn
Copy link
Member

LGTM, as its just an import

@EmmaKnijn EmmaKnijn added kind: import from external source Imports from an external source, like an unmaintained standards repository. status: reviews needed A proposal needs reviews. labels Mar 30, 2024
Copy link
Member

@tomodachi94 tomodachi94 left a comment

Choose a reason for hiding this comment

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

Lgtm, just some minor corrections and a few points that should probably be addressed.

Copy link

@ShreksHellraiser ShreksHellraiser left a comment

Choose a reason for hiding this comment

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

The formatting mistake in CCSMB-16 should be corrected, and see my comment on CCSMB-15. Once the formatting mistake is corrected I think it is good to merge.

@znepb znepb requested a review from ShreksHellraiser April 3, 2024 22:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind: import from external source Imports from an external source, like an unmaintained standards repository. status: reviews needed A proposal needs reviews.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants