-
Notifications
You must be signed in to change notification settings - Fork 40
Remove docs directory and update README #2127
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
Conversation
f739d1c to
64ccd80
Compare
josh-wong
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've added some comments and suggestions. PTAL!
Co-authored-by: Josh Wong <joshua.wong@scalar-labs.com>
josh-wong
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thank you!🙇♂️
Torch3333
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you!
komamitsu
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! 👍
| - name: Build Getting Started project | ||
| run: ./gradlew assemble | ||
|
|
||
| build-check-example-project-for-kotlin: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As I commented on scalar-labs/scalardb-samples#68 (comment), this isn't copied to scalardb-samples. Probably it would be great if we add equivalent build tests to scalardb-samples later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@komamitsu Yes. We should do that. Thanks.
@KodaiD Let's do that later.
kota2and3kan
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thank you!
feeblefakie
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thank you!
Description
This PR removes the
docsdirectory and updates README.Related issues and/or PRs
N/A
Changes made
docsdirectory.README.mdChecklist
Additional notes (optional)
After merging this PR, we will stop publishing the GitHub pages.
Release notes
N/A