-
Notifications
You must be signed in to change notification settings - Fork 443
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
Add contributing guidelines with how to build docs #1065
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1065 +/- ##
===========================================
- Coverage 42.61% 42.6% -0.02%
Complexity 746 746
===========================================
Files 359 359
Lines 9867 9867
Branches 1078 1078
===========================================
- Hits 4205 4204 -1
Misses 5347 5347
- Partials 315 316 +1
Continue to review full report at Codecov.
|
* Adding a new datatype | ||
* Adding a new typeclass | ||
* Running tests | ||
* [Building documentation](#building-documentation) |
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.
Oh I like this.
CONTRIBUTING.md
Outdated
Documentation is being built by [ΛNK](https://github.com/arrow-kt/ank) which gives an ability to verify and interpret code snippets in the documentation. For example: | ||
|
||
|
||
```kotlin:ank |
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.
Extra spaces here
CONTRIBUTING.md
Outdated
To build the documentation, in the root of the project, run | ||
|
||
``` | ||
$ ./gradlew clean :arrow-docs:clean :arrow-docs:runAnk |
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.
clean implies arrow-docs:clean AFAIK.
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.
Nice first contribution, many thanks! This'll be useful :D Could you please fix the couple of tweaks I suggested before we merge?
@iboss-ptk I added the changes myself, so we could get it merge today :D Thanks for the first contribution! |
#1036