Skip to content

Document the use of mandatory macro parameters and add named macro tokens#1212

Merged
jmchilton merged 6 commits intogalaxyproject:masterfrom
bernt-matthias:topic/extend-macros-docs
Sep 9, 2022
Merged

Document the use of mandatory macro parameters and add named macro tokens#1212
jmchilton merged 6 commits intogalaxyproject:masterfrom
bernt-matthias:topic/extend-macros-docs

Conversation

@bernt-matthias
Copy link
Collaborator

learned while working on galaxyproject/galaxy#13152

@bernt-matthias
Copy link
Collaborator Author

I hope that I have addressed all comment.

Just had the idea that one could allow content to yield tags and interpret it as default if the corresponding token is not given.
Seems that writing docs is not only annoying but may result in better ideas and helps to reason about code.

@bernt-matthias bernt-matthias changed the title document the use of mandatory macro parameters Document the use of mandatory macro parameters and add named tokens Jan 28, 2022
@bernt-matthias bernt-matthias changed the title Document the use of mandatory macro parameters and add named tokens Document the use of mandatory macro parameters and add named macro tokens Jan 28, 2022
@bernt-matthias bernt-matthias force-pushed the topic/extend-macros-docs branch from 72dd9dc to 287de30 Compare April 20, 2022 17:02
@bernt-matthias
Copy link
Collaborator Author

Hey @nsoranzo there were quite a few commits after your approval. Are you still OK with merging?

bernt-matthias and others added 2 commits July 27, 2022 10:29
Co-authored-by: Nicola Soranzo <nicola.soranzo@gmail.com>
@jmchilton jmchilton merged commit 65450c7 into galaxyproject:master Sep 9, 2022
@jmchilton
Copy link
Member

Sorry for the delay on the merge - very nice work. Thank you both!

@bernt-matthias bernt-matthias deleted the topic/extend-macros-docs branch September 10, 2022 11:24
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.

4 participants