-
Notifications
You must be signed in to change notification settings - Fork 6
init/paper joss #211
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
init/paper joss #211
Conversation
I've fixed the action fail issue in another PR, I will rebase it after Sadra approves that. Now we can work in parallel and fulfill our sections in this PR (as we previously divided on hack.md) I spent a considerable time on the summary and Statement of the need parts, but still they are not the final version and I still need to tweak them a little bit more. Please work on your parts and let me know if you need my contribution anywhere. |
Hi @AHReccese, can you please break paragraphs into multi-lines? that way we can leave comments easier. |
Add `@`, refactor sentence beginning
Dear Sepand, I've applied your feedback regarding Let me know if you have anything else to apply. |
@sepandhaghighi |
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.
🚀
Reference Issues/PRs
What does this implement/fix? Explain your changes.
Any other comments?