Skip to content

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.

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

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

6 participants