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

Cleanup #150

Merged
merged 4 commits into from
Aug 2, 2023
Merged

Cleanup #150

merged 4 commits into from
Aug 2, 2023

Conversation

mikeknep
Copy link
Contributor

@mikeknep mikeknep commented Aug 2, 2023

  • Drop a few stale CSVs (all our examples under notebooks/ are referencing public S3 files) and a weird checked-in-but-empty directory
  • Remove linting make targets (this repo will inherit our internal, common styling rules shortly)
  • Remove relational README (prefer our public docs)

We should consider giving the main, root-level README an overhaul at some point, but I'm not going to get into that right now.

@mikeknep mikeknep requested a review from pimlock August 2, 2023 14:18
@mikeknep mikeknep merged commit 1c0ac47 into main Aug 2, 2023
@mikeknep mikeknep deleted the cleanup branch August 2, 2023 17:52
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