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.

Sign up for GitHub

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

Jump to bottom

datastore: split out formatting & sanity checks in their own modules #1625

Merged
merged 2 commits into from Mar 20, 2023
Merged

datastore: split out formatting & sanity checks in their own modules #1625

merged 2 commits into from Mar 20, 2023

Conversation

teh-cmc
Copy link
Member

@teh-cmc teh-cmc commented Mar 20, 2023 •
edited

Make store.rs readable again. No funny business, just shuffling code around.

Part of #1619

@teh-cmc teh-cmc added enhancement New feature or request 🧑‍💻 dev experience developer experience (excluding CI) ⛃ re_datastore affects the datastore itself labels Mar 20, 2023
@emilk emilk merged commit 28b526e into main Mar 20, 2023
@emilk emilk deleted the cmc/datastore/split_files branch March 20, 2023 16:06
@emilk emilk mentioned this pull request Mar 27, 2023
Merged
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@emilk emilk emilk approved these changes

Assignees
No one assigned
Labels
🧑‍💻 dev experience developer experience (excluding CI) enhancement New feature or request ⛃ re_datastore affects the datastore itself
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@teh-cmc @emilk