Skip to content

Feature template update#191

Merged
planes merged 4 commits intodevelopfrom
feature-template_update
Aug 14, 2017
Merged

Feature template update#191
planes merged 4 commits intodevelopfrom
feature-template_update

Conversation

@timdmacdo
Copy link
Contributor

This adds documentation syntax to the templates and removes high level documentation from the doxygen page (it will be added to the main website instead)

@coveralls
Copy link

Coverage Status

Coverage remained the same at 55.312% when pulling db96947 on feature-template_update into 75998b1 on develop.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 55.312% when pulling 1ff1851 on feature-template_update into 75998b1 on develop.

@planes
Copy link
Member

planes commented Aug 14, 2017

This looks good. The only comment I would say is that it might be nice to add an example of how you like to handle quantities with units in the inputs and outputs on the templates.

@planes
Copy link
Member

planes commented Aug 14, 2017

Looks good.

@planes planes merged commit 9337076 into develop Aug 14, 2017
@coveralls
Copy link

Coverage Status

Coverage remained the same at 55.312% when pulling 54ddbd5 on feature-template_update into 75998b1 on develop.

@planes planes deleted the feature-template_update branch August 15, 2017 23:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants