-
Notifications
You must be signed in to change notification settings - Fork 6
Add Compass report summary #4
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
Conversation
|
Thanks @nikhilwoodruff, good to see this summary. Could it be merged with the other simulations in #3? If so, it might be easier to start with a Google Doc so you and @bkh248 can collaborate on the content, then focus on getting it into Markdown. Or you could try https://stackedit.io for collaborative Markdown (I haven't used this). |
|
Hello @nikhilwoodruff! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
|
|
Er, didn't intend to update this PR, but it's pretty old, so will close and make a new one. |
Just opening this with a summary of the Compass report, feedback welcome (too long/too short, headers/no headers?).