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

Return completionItem documentation as plain text #238

Merged
merged 1 commit into from
Jul 24, 2017

Conversation

fbricon
Copy link
Contributor

@fbricon fbricon commented May 26, 2017

Fixes redhat-developer/vscode-java#215

Signed-off-by: Fred Bricon fbricon@gmail.com

@fbricon fbricon requested a review from gorkem May 26, 2017 16:05
import org.junit.Test;

/**
* Tests Javadoc to Markdown conversion
Copy link
Contributor

Choose a reason for hiding this comment

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

Change markdown to plain text?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

good catch

@@ -0,0 +1,114 @@
/**
* Copyright © 2009-2017, Jonathan Hedley <jonathan@hedley.net>
Copy link
Contributor

Choose a reason for hiding this comment

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

Are we able to add this file without a CQ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I wasn't expecting to get away with it. We'll need to CQ JSoup anyway (dependency of Remark)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

we're approved to check in

Fixes redhat-developer/vscode-java#215

Signed-off-by: Fred Bricon <fbricon@gmail.com>
@fbricon fbricon force-pushed the plaintext-descriptions branch from e8df70f to a522ae1 Compare July 12, 2017 20:00
@fbricon fbricon requested review from snjeza and aeschli July 23, 2017 06:22
@fbricon fbricon merged commit 2fd722f into eclipse-jdtls:master Jul 24, 2017
@fbricon fbricon deleted the plaintext-descriptions branch September 20, 2017 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants