Skip to content
This repository has been archived by the owner on Aug 7, 2020. It is now read-only.

Allow to add @since tag based on package.json #193

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

KevinGuancheDarias
Copy link

Hello, in my projects it is mandatory to include since tags in all public elements.

So I think it's useful if the Document This extension does this by itselt.

Notice @joelday : It includes a new dependency, maybe you know another way to do it purely with VSCode extensions framework

@joelday
Copy link
Owner

joelday commented Feb 14, 2019

My apologies for the delay. Can you please catch up with master and test this for any regressions? Thank you!

@KevinGuancheDarias
Copy link
Author

No problem, Have been using it in my projects without any issue for the last two months.

Works only in classes and methods, I didn't found how to hook into the public property definition.

Thanks for your great work in this extension!

@TheOpenDevProject
Copy link
Collaborator

@KevinGuancheDarias If you want this merged please fix your merge conflicts and Ill test it for regressions.

Merge branch 'master' into us-763a83f
@KevinGuancheDarias
Copy link
Author

Hello @TheOpenDevProject the merge conflicts have been solved, srry for the delay!.

Best regards! 🍷

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants