eclipse-cs icon indicating copy to clipboard operation
eclipse-cs copied to clipboard
verified publisher icon checkstyle

add "Open Rule link in browser"

Open qxo opened this issue 5 years ago • 4 comments

This PR will index the checks.html content, link rule to the detail url. ie:

AbstractClassName -->  https://checkstyle.sourceforge.io/config_naming.html#AbstractClassName

we can change the rule info url：

-Declipsecs_rules_url=https://checkstyle.sourceforge.io/checks.html

qxo avatar Sep 13 '20 03:09 qxo

CI failures https://travis-ci.org/github/checkstyle/eclipse-cs/jobs/727087072#L962 please fix

romani avatar Sep 14 '20 19:09 romani

@qxo , you will not need to cache anything ones https://github.com/checkstyle/eclipse-cs/pull/257 is merged. Please help to test this PR, it will simplify a lot your PR and any other problems.

romani avatar Sep 19 '20 14:09 romani

@qxo , you will not need to cache anything ones #257 is merged. Please help to test this PR, it will simplify a lot your PR and any other problems.

ok, if I had the time.

What's the #257 problem which we should focus on?

qxo avatar Sep 19 '20 15:09 qxo

What's the #257 problem which we should focus on?

we try to get all data about Checks from checkstyle library itself as checkstyle jar now contain almost all metadata(all docs) that is required for plugin and users.

romani avatar Sep 19 '20 15:09 romani

I'd like to close this in favour of #568. I'd avoid trying to index any online URL to get the available help URLs. We could eventually resurrect the "Open URL" actions from this PR.

Bananeweizen avatar Sep 19 '23 17:09 Bananeweizen