Skip to content

Fix issue #33, update #21 - #37

Merged
azu merged 2 commits into
azu:masterfrom
gdolle:fix-#33-#21
Jan 6, 2017
Merged

Fix issue #33, update #21#37
azu merged 2 commits into
azu:masterfrom
gdolle:fix-#33-#21

Conversation

@gdolle

@gdolle gdolle commented Jan 5, 2017

Copy link
Copy Markdown
Contributor

Fix issue #33 and update for #21

  • Fix readme
  • Add python support comments (## [tag])
  • Implement the workaround for C++ (replace c_cpp by cpp) for now. (This should be improved in the future)
  • Add specific C++ test.
  • Add python comment ## test

gdolle added 2 commits January 5, 2017 23:33
@gdolle
Fix issue azu#33
013189b 
- Add bash,python, ... comment style support
- Add corresponding test
- Fix readme
The readme was wrong, it is not "triple char", but "doxygen documenting
code" standard (NB: comment style per programming language).
@gdolle
Update issue azu#21
423851e 
- Implement workaround for C++
- Add new test to check code "cpp" tag instead of "c_cpp"
- remove cpp still buggy test (for now..)
@gdolle

gdolle commented Jan 5, 2017

Copy link
Copy Markdown
Contributor Author

Hi @azu, the #33 PR up ;)

@azu

azu commented Jan 6, 2017

Copy link
Copy Markdown
Owner

@gdolle Good! Nice work.

@azu
azu merged commit c916292 into azu:master Jan 6, 2017
@azu azu mentioned this pull request Jan 6, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@gdolle @azu