Skip to content

README: add spx reader#125

Closed
kaalleen wants to merge 1 commit intomainfrom
kaalleen/readme-spx-reader
Closed

README: add spx reader#125
kaalleen wants to merge 1 commit intomainfrom
kaalleen/readme-spx-reader

Conversation

@kaalleen
Copy link
Collaborator

@kaalleen kaalleen commented Aug 2, 2025

... the file format was missing although a reader seems to exist ...

@kaalleen kaalleen requested a review from not-my-profile August 2, 2025 12:03
@not-my-profile
Copy link
Collaborator

I think we should autogenerate these lists in the README and have a test that checks they're up-to-date.

@kaalleen
Copy link
Collaborator Author

kaalleen commented Aug 3, 2025

Sounds very reasonable. It's a bit out of the scope of this pull request though.

@not-my-profile
Copy link
Collaborator

Implemented in #127.

@kaalleen
Copy link
Collaborator Author

kaalleen commented Aug 3, 2025

Closing in favor of #127

@kaalleen kaalleen closed this Aug 3, 2025
@kaalleen kaalleen deleted the kaalleen/readme-spx-reader branch August 3, 2025 07:36
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