Breaking: remove attachComment option

TSC Summary: Now that ESLint does not use the attachComment option and given its buggy nature and maintenance cost, I think we should remove the option from Espree.

TSC Question: This is a breaking change, however I suspect that very few other projects are using this option.

I’m more than happy to do the work if this is accepted.

[kaicataldo]

@benmosher You can still use methods on the SourceCode object returned by context.getSourceCode() to find comments based on their location relative to a given node or token (namely getCommentsBefore(nodeOrToken), getCommentsAfter(nodeOrToken), and getCommentsInside(node)). See the docs here for more details and feel free to stop by the Gitter if something isn’t clear!

[platinumazure]

@nfroidure espree still parses comments- they’re just stored in a comments array rather than attached directly to nodes. You can iterate through all comments pretty easily. Looking for comments “near nodes” is a bit harder, but still possible: If you are using range info on nodes (and tokens and comments), you could get all comments that start before/after the node you’re looking at and see all comments that might have been attached to the node in the old comment attachment strategy.

You could also look at ESLint’s token store utilities to get an idea of how ESLint manages this information internally.