-
Notifications
You must be signed in to change notification settings - Fork 63
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
new annotations #70
base: master
Are you sure you want to change the base?
new annotations #70
Conversation
- added warning with badge - added constraint - added require - added feature with badge - added deprecated with badge - added trap - added env - updated @example allowing to change default script language
Glad to see |
Thanks for the suggestion, actually it could be easily to implement it using a different style, as I implemented it in github style, it makes sense to implement it as a github badge. Maybe you could create your own style starting from this branch. |
When testing this change, I had trouble keeping the badge for # @name Project Name
# @brief Brief
# @description overview
# @description My super function. Not thread safe.
# @deprecated use some other thing instead
b() {
:
} results in: # Project Name
Brief
## Overview
overview
## Index
* [b](#b)
### b
My super function. Not thread safe.
@deprecated use some other thing instead |
Thanks to have reported it with a reproductible test case, I fixed it, you can check my commit |
Hello,
first thanks for your project
I added some features with unit tests included
new annotations
- added @warning with badge
- added @constraint
- added @require
- added @feature with badge
- added @deprecated with badge
- added @trap
- added @env
The badge feature forced me to add an urlencode function to construct valid links
One little change
- updated @example allowing to change default script language
I also updated README.md example, removing the table because actually I discovered the rendered file viewing at source code.
The scrollbar on the table on github website was hiding the comparison.
Do not hesitate to give me your feedbacks