Docblockr wrap lines error about docblockr HOT 18 CLOSED

Got an identical error, but on line 487 of docblockr-worker.js.

from docblockr.

@tnbeatty I dont think you will get an error on line 487, as that line is an entirely different function, that will not be called for wrap lines. Please check and get back on this

from docblockr.

@ilanbiala @tnbeatty Commit 3e68d29 should fix this. Will close this for now, let me know if this repeats

from docblockr.

Got the error again, this time on line 562, running v0.5.7.

from docblockr.

@ilanbiala Can you please check against the new release and post the error message with an example.. it would help me solve it easier... Thank u

from docblockr.

@nikhilkalige tested against 0.5.8. Atom doesn't seem to report that Docblockr has an update, maybe you should also investigate that.

Here's a screenshot:

from docblockr.

@nikhilkalige any thoughts on the error?

from docblockr.

@ilanbiala Sorry.. havent looked at it.. college.. lots of work :(

from docblockr.

Pull request has fix

from docblockr.

@nikhilkalige take a look at #28 please.

from docblockr.

@nikhilkalige is this going to be fixed?

from docblockr.

With latest atom and docblocker version seems already resolved. Can you confirm there's no more problem about this plz ?

from docblockr.

still an issue for me.

from docblockr.

/**
 * Keeping packages up to date helps keep things speedy and breakage to a minimum. Upgrade often! Keeping packages up to date helps keep things speedy and breakage to a minimum. Upgrade often!
 * Keeping packages up to date helps keep things speedy and breakage to a minimum. Upgrade often! Keeping packages up to date helps keep things speedy and breakage to a minimum. Upgrade often!
 * Keeping packages up to date helps keep things speedy and breakage to a minimum. Upgrade often! Keeping packages up to date helps keep things speedy and breakage to a minimum. Upgrade often!
 */

Docblockr: Wrap Lines

/**
 * Keeping packages up to date helps keep things speedy and breakage to a
 * minimum. Upgrade often! Keeping packages up to date helps keep things
 * speedy and breakage to a minimum. Upgrade often! Keeping packages up to
 * date helps keep things speedy and breakage to a minimum. Upgrade often!
 * Keeping packages up to date helps keep things speedy and breakage to a
 * minimum. Upgrade often! Keeping packages up to date helps keep things
 * speedy and breakage to a minimum. Upgrade often! Keeping packages up to
 * date helps keep things speedy and breakage to a minimum. Upgrade often!
 */

Can you put up an example?

from docblockr.

I get an error when I try to wrap a multiline decorated comment. That case isn't handled.

from docblockr.

Ok confirmed on decorated comments here, started to work on this one not so trivial i'll have another look later.
For now i spotted some exp which need rework to get this working for multiline:

in docblockr-worker.js:
line 556:

_col = _text.search(/^\s*\*(\/)/);
// should become
_col = _text.search(/^\s*\*+(\/)/);

line 577 is not made to work with multiline comments either.

finally even getting the two previously mentioned ok it still need to reduce first and ending decoration line length which i had no time to get a look in.

will check again another time.

from docblockr.

This seems like a pretty big issue - still experiencing it.

from docblockr.

Fixed by #222. Released with version v0.8.7.

from docblockr.