Skip to content
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

Allow @see to pass markdown links as is. Close #52. #60

Merged
merged 3 commits into from
Nov 7, 2022

Conversation

landure
Copy link
Contributor

@landure landure commented Oct 27, 2022

This modification detect is @see contains a markdown link ([]()) and pass the content as is if it is the case.

This close #52 and allow to use HTTP(S) URLs in @see.

I've updated the readme, the example file and unit tests.

@kovetskiy
Copy link
Member

Would be great to have tests for this as well

@landure
Copy link
Contributor Author

landure commented Oct 27, 2022

I'll do what i can to add test. Do you have some requests on what test to add ? I've already updated the main function-tags test.

@landure
Copy link
Contributor Author

landure commented Oct 27, 2022

Hi, i've added some functionnal tests and enhanced the @see functionnality to detect URL in text, and added a basic support for paths starting by '/', './' and '../'.

@kovetskiy kovetskiy merged commit 00d00b9 into reconquest:master Nov 7, 2022
@kovetskiy
Copy link
Member

@landure Hey! Thank you for all your contributions, I've merged all of your PRs.

landure added a commit to landure/shdoc that referenced this pull request Aug 17, 2023
…uest#60)

* Allow @see to pass markdown links as is. Close reconquest#52.

* Enhance @see functionnality with URL detection and add functionnal tests.

Co-authored-by: Egor Kovetskiy <e.kovetskiy@gmail.com>
landure added a commit to landure/shdoc that referenced this pull request Aug 17, 2023
…uest#60)

* Allow @see to pass markdown links as is. Close reconquest#52.

* Enhance @see functionnality with URL detection and add functionnal tests.

Co-authored-by: Egor Kovetskiy <e.kovetskiy@gmail.com>
@landure landure deleted the see-links branch August 19, 2023 09:47
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.

Add support for cross documentation @see links
2 participants