-
Notifications
You must be signed in to change notification settings - Fork 0
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
Initial data class commit. #29
Conversation
Going to pull this branch down to the RSP and test it before I request a review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you remove the .DS_Store files and add .DS_Store to the .gitignore file - it's empty mac files and so we should not include them in the repository
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, I think the data structure has everything I need for now. I agree with Meg on gitignoring local .DS_Store files
Good catch. I'd assumed .DS_Store was in the .gitignore. That should be it now. |
Still need to remove the .DS_store that already got pushed before this PR. Those still exist in the PR |
Argh. Sorry. That should be all of them now. |
Fixes #27.
This PR begins the basic data class structure for Adler.
Fixes #28.
Adds a command line script so Adler can be called on the command line. Changed the file structure a bit.
I have tested that this all works on the RSP. Yay!