Included documentation of secondary files#318
Included documentation of secondary files#318Mackenzie-OO7 wants to merge 28 commits intocommon-workflow-language:mainfrom
Conversation
kinow
left a comment
There was a problem hiding this comment.
Quick review during lunch break 🏃♂️ Thanks @Mackenzie-OO7 !
|
@kinow Thanks for reviewing. I've implemented the changes you suggested. |
Hi all - My only concern is 2 things Would be valid in this case |
|
@tetron can you give your take on secondary files and then merge when we resolve the issue? |
|
@swzCuroverse I've made some changes, is it okay now? |
|
@Mackenzie-OO7 could you give this pull request a more descriptive title? The PR title does not indicate that you are added documentation about secondary files. |
Alright. Is it better now? |
Yes, thank you! |
|
@tetron I've made the changes requested. |
|
Happy to get you back working on this. I believe the issues is that you are describing what the parser does more than what the user will do in the last section --- Let me make some suggestions on the text. |
This PR: