Skip to content

Add documentation website#112

Merged
olafurpg merged 4 commits into
scip-code:mainfrom
olafurpg:documentation-website
Mar 5, 2021
Merged

Add documentation website#112
olafurpg merged 4 commits into
scip-code:mainfrom
olafurpg:documentation-website

Conversation

@olafurpg

@olafurpg olafurpg commented Mar 4, 2021

Copy link
Copy Markdown
Contributor

Previously, lsif-java didn't have any documentation for end-users. We
only had documentation for contributors. This commit adds a website that
includes user-facing documentation on how to run lsif-java with
automatic indexing and manual indexing.

@olafurpg

olafurpg commented Mar 4, 2021

Copy link
Copy Markdown
Contributor Author

Live preview at https://sourcegraph.github.io/lsif-java

Just opening this PR to close my day and collect early feedback. I will review the change again tomorrow and likely clean up a few small things.

@olafurpg olafurpg requested a review from Strum355 March 4, 2021 20:00
Previously, lsif-java didn't have any documentation for end-users. We
only had documentation for contributors. This commit adds a website that
includes user-facing documentation on how to run lsif-java with
automatic indexing and manual indexing.
@olafurpg olafurpg force-pushed the documentation-website branch from a8515fe to 0b8f679 Compare March 4, 2021 20:05
Comment thread docs/getting-started.md
Comment on lines +127 to +133
| Build tool | Support | Tracking issue |
| ---------- | ------- | -------------------------------------------------------------------------------- |
| Gradle | ✅ | |
| Maven | ✅ | |
| Bazel | ❌ | [sourcegraph/lsif-java#88](https://github.com/sourcegraph/lsif-java/issues/88) |
| Buck | ❌ | [sourcegraph/lsif-java#99](https://github.com/sourcegraph/lsif-java/issues/99) |
| sbt | ❌ | [sourcegraph/lsif-java#110](https://github.com/sourcegraph/lsif-java/issues/110) |

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

image

Comment thread docs/manual-configuration.md Outdated
Comment thread docs/manual-configuration.md Outdated
Comment thread docs/manual-configuration.md Outdated

![A three stage pipeline that starts with a list of Java sources, creates a list of SemanticDB files that then become a single LSIF index.](docs/img/semanticdb-javac-pipeline.svg)

## Step 1: Add SemanticDB compiler plugin to the classpath

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should we have consistent first letter capitalization or is this intentionally capitalized?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I love your attention to details 👌 fixed

@Strum355 Strum355 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Really like having this website, great idea 🎉
Some minor but not-really-blocking comments 🙂

@olafurpg olafurpg left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thank you for the review!

Comment thread docs/manual-configuration.md Outdated

![A three stage pipeline that starts with a list of Java sources, creates a list of SemanticDB files that then become a single LSIF index.](docs/img/semanticdb-javac-pipeline.svg)

## Step 1: Add SemanticDB compiler plugin to the classpath

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I love your attention to details 👌 fixed

Comment thread docs/manual-configuration.md Outdated
Comment thread docs/manual-configuration.md Outdated
@olafurpg olafurpg force-pushed the documentation-website branch from 65c959b to 387826b Compare March 5, 2021 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants