Skip to content
This repository was archived by the owner on Mar 23, 2024. It is now read-only.

Update jscs-jsdoc to 1.3.0#2009

Closed
ljharb wants to merge 1 commit intojscs-dev:masterfrom
ljharb:update_jsdoc
Closed

Update jscs-jsdoc to 1.3.0#2009
ljharb wants to merge 1 commit intojscs-dev:masterfrom
ljharb:update_jsdoc

Conversation

@ljharb
Copy link
Copy Markdown
Contributor

@ljharb ljharb commented Dec 8, 2015

This includes jscs-dev/jscs-jsdoc#179 which includes syavorsky/comment-parser#24 (comment). Since this PR also properly includes a semver range identifier, fixes of this type (for comment-parser and jscs-jsdoc) should no longer require corresponding PRs across the dependency graph :-)

This will hopefully make jscs work on node 0.8 (that doesn't mean you have to officially support it!) and if not, will at least remove one bottleneck so I can pursue the others.

jscs is included because jscs-jsdoc apparently now requires it as a peerDep.

@markelog
Copy link
Copy Markdown
Member

markelog commented Dec 8, 2015

jscs is included because jscs-jsdoc apparently now requires it as a peerDep.

So to be dependant on itself o_0?

@markelog
Copy link
Copy Markdown
Member

markelog commented Dec 8, 2015

Okay, talked with @zxqfox, we decided to remove that peer dep, we will land this right after the release of the jscs-jsdoc 1.3.1 and push it up with jscs 2.7.

@ljharb
Copy link
Copy Markdown
Contributor Author

ljharb commented Dec 8, 2015

Awesome, that sounds great! I don't have an opinion on the peerDep, I just want 1.3.0 or higher :-D

qfox added a commit to jscs-dev/jscs-jsdoc that referenced this pull request Dec 8, 2015
@qfox qfox closed this in 12e8fb5 Dec 8, 2015
@ljharb ljharb deleted the update_jsdoc branch December 8, 2015 23:48
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.

2 participants