-
Notifications
You must be signed in to change notification settings - Fork 1
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
First version of PAS Explanatory Report added #41
First version of PAS Explanatory Report added #41
Conversation
@mbeckerle I have added comments to the places I was tagged. |
Last changes are in. @smhdfdl with an official approval from you on this review I will squash 6 commits to one coherent commit and merge back to the master branch and let others know to submit it. |
@smhdfdl I believe I fixed these additional issues now. |
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.
LGTM
This is written in markdown. Review was in OpenGridForum#41
d195256
to
5ec23bf
Compare
This is written in markdown. Review was in OpenGridForum#41
This is in markdown.
Use new line for each new sentence style. Long sentences can have multiple lines, but each new sentence should start on a new line.