Skip to content

Chore/jsdoc4 update#242

Open
njetsy wants to merge 12 commits into
SoftwareBrothers:masterfrom
njetsy:chore/jsdoc4-update
Open

Chore/jsdoc4 update#242
njetsy wants to merge 12 commits into
SoftwareBrothers:masterfrom
njetsy:chore/jsdoc4-update

Conversation

@njetsy

@njetsy njetsy commented Feb 25, 2023

Copy link
Copy Markdown

Some updates to improve compatibility with current versions of packages:

This probably needs some more work or another look over but it should be a good start. I did not do any vue testing, but it is working with the latest version of parcel 2, react 18, and jsdoc 4. Both category and component tags confirmed working with live previews and prop-type recognition.

(Thanks for the amazing head start with this BTW!)

@njetsy

njetsy commented Mar 2, 2023

Copy link
Copy Markdown
Author

Added in fix for #174. Subcategory now works for globals.

@nxame nxame left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@njetsy everything looks good to me.

NAB: Some files formatting is odd. Eg. publish.js Line 372, react-wrapper.js Line 74.

@nxame

nxame commented Mar 7, 2023

Copy link
Copy Markdown

@wojtek-krysiak Can anyone help us here? This PR would really be welcome by the community since it fixes so many issues.

@pacholoamit

Copy link
Copy Markdown

Yeah, this PR defs fixes issues I've been having with better-docs

@wmurphyrd

Copy link
Copy Markdown

Another vote that this works. Installed this version with jsdoc 4.0.2 and the audit issues are resolved while my docs still work the same

@par5ul1

par5ul1 commented Aug 8, 2023

Copy link
Copy Markdown

Any progress on this?

@dziraf

dziraf commented Dec 7, 2023

Copy link
Copy Markdown

@njetsy @par5ul1 @wmurphyrd @pacholoamit @absqueued

Hi, I've just released this on new @dev channel:
https://github.com/SoftwareBrothers/better-docs/releases/tag/v2.7.4-dev.1
There's no changelog available because semantic-release relies on feat: and fix: prefixed commits to build it. I'll fix it later when merging this PR to master. I haven't been able to test these changes, that's why it's prereleased for now.
Keep in mind you can have issues with @component tag since it's unit tests failed and I temporarily disabled them to force this release. I'll try to play around with this library and hopefully bring it back to life soon...

You can install this via yarn add better-docs@dev.

The old beta prerelease channel will be archived soon. It diverged too much from the master branch for some reason...

@njetsy

njetsy commented Dec 8, 2023

Copy link
Copy Markdown
Author

Thank you so much for the update!! I'll give the new version a go.

@njetsy

njetsy commented Dec 9, 2023

Copy link
Copy Markdown
Author

@dziraf

Thank you again for releasing this! Everything is working perfectly in my setup.

As an FYI, I've only run into two issues when using the @component tag so far:

  1. If a @component renders a react-router <Outlet />, the build crashes.
  2. If there is no default export in a file, the build crashes.

In both cases I think parcel spits out an error during the bundling process.

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.

6 participants