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
Copy Markdown

@znepb znepb commented Mar 30, 2024

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

Copy link
Copy Markdown
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

Comment thread Standards/CCSMB-15.md Outdated
Comment thread Standards/CCSMB-16.md Outdated
@EmmaKnijn
Copy link
Copy Markdown
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
Copy Markdown
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.

Comment thread Standards/CCSMB-15.md Outdated
Comment thread Standards/CCSMB-15.md Outdated
Comment thread Standards/CCSMB-15.md
Comment thread Standards/CCSMB-15.md
znepb and others added 3 commits March 30, 2024 19:19
Co-authored-by: Tomo <68489118+tomodachi94@users.noreply.github.com>
Co-authored-by: Tomo <68489118+tomodachi94@users.noreply.github.com>
Copy link
Copy Markdown

@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.

Comment thread Standards/CCSMB-15.md
Comment thread Standards/CCSMB-16.md Outdated
@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