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

Prevent jsdoc upgrade #1349

Closed

Conversation

schiffermtq
Copy link
Contributor

pbts currently doesn't work with jsdoc 3.6.x and will emit broken
definitions when trying to use it.

As the toplevel package-lock.json is not respected by npm install as
executed by the CLI utils, fix the jsdoc version to 3.5.5 in
package.json.

Fixes #1342

pbts currently doesn't work with jsdoc 3.6.x and will emit broken
definitions when trying to use it.

As the toplevel package-lock.json is not respected by `npm install` as
executed by the CLI utils, fix the jsdoc version to 3.5.5 in
package.json.

Fixes protobufjs#1342
dcodeIO
dcodeIO previously approved these changes Feb 4, 2020
Copy link
Member

@dcodeIO dcodeIO left a comment

Choose a reason for hiding this comment

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

LGTM, unless there's a better fix after upgrading jsdoc. Perhaps #1338 - or #1333?

@schiffermtq
Copy link
Contributor Author

Hmm, the Node 12 incompatibility is unfortunate.

@dcodeIO dcodeIO dismissed their stale review February 5, 2020 11:32

superseded by #1356

@schiffermtq schiffermtq deleted the fix/prevent-jsdoc-upgrade branch October 13, 2021 12:28
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.

pbts doesn't generate enum typings when the enum is defined after its use
2 participants