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

Change Comments from /*! to /** to make possible jsdoc available #12337

Merged
merged 4 commits into from Aug 30, 2022

Conversation

hasezoey
Copy link
Collaborator

Summary

This PR changes most /*! to /** to make jsdoc available for those that have descriptions / tags defined, also does:

  • change some casing for * ignore
  • change some @ignore to ignore
  • change some comments from /*! to //
  • add / change tags to be more in line with current implementations (for those that were /*! before)
  • change all @private to @api private
  • changes api public to api private for const mongoose = module.exports = exports = new Mongoose({, because this had no effect anyway (from what i could tell)
  • Model.prototype.$remove is now public in the documentation
  • Model.db; now properly appears as a static property in the documentation
  • ArraySubdocument.prototype.$parent is now api public (because other aliases are)
  • removes @api tags from jsdoc in test/* files

Things that were ignored:

  • all that were just ignore
  • all that said module exports
  • all that said inherits from (assigning .prototype / ._proto_)

this makes the all description and types available via jsdoc, but still excluded from the documentation
Copy link
Collaborator

@vkarpov15 vkarpov15 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

lib/types/ArraySubdocument.js Outdated Show resolved Hide resolved
@vkarpov15 vkarpov15 added this to the 6.5.4 milestone Aug 30, 2022
@vkarpov15
Copy link
Collaborator

Also, I'm sorry but it looks like I broke tests by resolving merge conflict. Any ideas on how to fix @hasezoey ?

@hasezoey
Copy link
Collaborator Author

Also, I'm sorry but it looks like I broke tests by resolving merge conflict. Any ideas on how to fix @hasezoey ?

i still have it locally, i will force-push a fix

@hasezoey
Copy link
Collaborator Author

fixed with proper merge

@vkarpov15 vkarpov15 merged commit 67bfddd into Automattic:master Aug 30, 2022
@hasezoey hasezoey deleted the changeCommentType branch August 30, 2022 16:07
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.

None yet

2 participants