Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: remove schema compilation from build process #33

Merged
merged 3 commits into from
Feb 17, 2023

Conversation

ev3nvy
Copy link
Owner

@ev3nvy ev3nvy commented Feb 17, 2023

Description

This changes the build process so that it no longer compiles the flatbuffers schema using the build script. Instead, we now just have the compiled schema inside of src/generated folder. This closes #31 because we no longer have to ship flatc alongside out project.

Motivation

This way we don't need to ship flatc with the project.

This commit removes schema compilation, but instead uses a schema from
the releases section over at `rman-schema`. Submodule `rman-schema` is
no longer being used, however it will stay linked and updated as a way
to track which version of the schema is being used.
@ev3nvy ev3nvy added type: chore Updating grunt tasks status: completed Issue was resolved, or pull request was merged labels Feb 17, 2023
@ev3nvy ev3nvy added this to the v0.2.0 milestone Feb 17, 2023
Once `format_generated_files` is stabilised we can use that instead.
@ev3nvy ev3nvy merged commit 76f75e3 into master Feb 17, 2023
@ev3nvy ev3nvy deleted the remove-schema-compilation branch February 17, 2023 17:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: completed Issue was resolved, or pull request was merged type: chore Updating grunt tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Huge crate size on crates.io
1 participant