Skip to content

Commit

Permalink
Add CI for checking markdown format. (#325)
Browse files Browse the repository at this point in the history
Signed-off-by: ChenYing Kuo <[email protected]>
  • Loading branch information
evshary authored Dec 18, 2024
1 parent 5b7cf5d commit 70b8709
Show file tree
Hide file tree
Showing 4 changed files with 66 additions and 39 deletions.
11 changes: 10 additions & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -59,6 +59,15 @@ jobs:
- name: Gradle Test
run: gradle jvmTest --info

markdown_lint:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: DavidAnson/markdownlint-cli2-action@v18
with:
config: '.markdownlint.yaml'
globs: '**/README.md'

# NOTE: In GitHub repository settings, the "Require status checks to pass
# before merging" branch protection rule ensures that commits are only merged
# from branches where specific status checks have passed. These checks are
Expand All @@ -67,7 +76,7 @@ jobs:
ci:
name: CI status checks
runs-on: ubuntu-latest
needs: build
needs: [build, markdown_lint]
if: always()
steps:
- name: Check whether all jobs pass
Expand Down
7 changes: 7 additions & 0 deletions .markdownlint.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
{
"MD013": false, # Line length limitation
"MD024": false, # Allow multiple headings with the same content
"MD033": false, # Enable Inline HTML
"MD041": false, # Allow first line heading
"MD045": false, # Allow Images have no alternate text
}
66 changes: 35 additions & 31 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@
[![License](https://img.shields.io/badge/License-EPL%202.0-blue)](https://choosealicense.com/licenses/epl-2.0/)
[![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0)


# Eclipse Zenoh

The Eclipse Zenoh: Zero Overhead Pub/sub, Store/Query and Compute.
Expand All @@ -16,36 +15,34 @@ Zenoh (pronounce _/zeno/_) unifies data in motion, data at rest and computations

Check the website [zenoh.io](http://zenoh.io) and the [roadmap](https://github.com/eclipse-zenoh/roadmap) for more detailed information.


----

# <img src="kotlin-logo.png" alt="Kotlin" height="50"> Kotlin API


This repository provides a Kotlin binding based on the main [Zenoh implementation written in Rust](https://github.com/eclipse-zenoh/zenoh).

The code relies on the Zenoh JNI native library, which written in Rust and communicates with the Kotlin layer via the Java Native Interface (JNI).

## Documentation

The documentation of the API is published at https://eclipse-zenoh.github.io/zenoh-kotlin/index.html.
The documentation of the API is published [here](https://eclipse-zenoh.github.io/zenoh-kotlin/index.html).

Alternatively, you can build it locally as [explained below](#building-the-documentation).

----

# How to import

:warning: Note nº1: **About the released packages**

The released packages can be found in the "packages" section of this repository, or in the links below:
* [Android packages](https://github.com/eclipse-zenoh/zenoh-kotlin/packages/1968034/versions)
* [JVM packages](https://github.com/eclipse-zenoh/zenoh-kotlin/packages/2016733/versions)

- [Android packages](https://github.com/eclipse-zenoh/zenoh-kotlin/packages/1968034/versions)
- [JVM packages](https://github.com/eclipse-zenoh/zenoh-kotlin/packages/2016733/versions)

These are 'nightly' packages that need to be imported as explained in the following sections of this README.
We plan to publish our packages into Maven central in the very short term.



## <img src="android-robot.png" alt="Android" height="50"> Android

For this first version we have published a [Github package](https://github.com/eclipse-zenoh/zenoh-kotlin/packages/1968034) with the library which can be imported on your projects.
Expand Down Expand Up @@ -84,6 +81,7 @@ implementation("org.eclipse.zenoh:zenoh-kotlin-android:1.0.0")
### Platforms

The library targets the following platforms:

- x86
- x86_64
- arm
Expand All @@ -102,8 +100,9 @@ Zenoh is a communications protocol, therefore the permissions required are:
<uses-permission android:name="android.permission.ACCESS_NETWORK_STATE"/>
```

---
# <img src="jvm.png" alt="Java" height="50"> JVM
----

## <img src="jvm.png" alt="Java" height="50"> JVM

Similar to Android, we have published a Github package to import on your projects.

Expand Down Expand Up @@ -140,26 +139,27 @@ implementation("io.zenoh:zenoh-kotlin-jvm:1.0.0")

For the moment, the library targets the following platforms:

- x86_64-unknown-linux-gnu
- aarch64-unknown-linux-gnu
- x86_64-apple-darwin
- aarch64-apple-darwin
- x86_64-pc-windows-msvc

- x86_64-unknown-linux-gnu
- aarch64-unknown-linux-gnu
- x86_64-apple-darwin
- aarch64-apple-darwin
- x86_64-pc-windows-msvc

---
----

# How to build it

## What you need

Basically:
* Rust ([Installation guide](https://doc.rust-lang.org/cargo/getting-started/installation.html))
* Kotlin ([Installation guide](https://kotlinlang.org/docs/getting-started.html#backend))
* Gradle ([Installation guide](https://gradle.org/install/))

- Rust ([Installation guide](https://doc.rust-lang.org/cargo/getting-started/installation.html))
- Kotlin ([Installation guide](https://kotlinlang.org/docs/getting-started.html#backend))
- Gradle ([Installation guide](https://gradle.org/install/))

and in case of targetting Android you'll also need:
* Android SDK ([Installation guide](https://developer.android.com/about/versions/11/setup-sdk))

- Android SDK ([Installation guide](https://developer.android.com/about/versions/11/setup-sdk))

## <img src="jvm.png" alt="JVM" height="50"> JVM

Expand All @@ -179,7 +179,7 @@ Once we have published the package, we should be able to find it under `~/.m2/re

Finally, in the `build.gradle.kts` file of the project where you intend to use this library, add mavenLocal to the list of repositories and add zenoh-kotlin as a dependency:

```
```kotlin
repositories {
mavenCentral()
mavenLocal()
Expand All @@ -202,14 +202,14 @@ It can be set up by using Android Studio (go to `Preferences > Languages & Frame
or alternatively it can be found [here](https://developer.android.com/ndk/downloads).

The native platforms we are going to target are the following ones:
```

- x86
- x86_64
- arm
- arm64
```

Therefore, if they are not yet already added to the Rust toolchain, run:

```bash
rustup target add armv7-linux-androideabi; \
rustup target add i686-linux-android; \
Expand All @@ -219,8 +219,8 @@ rustup target add x86_64-linux-android

to install them.


So, in order to publish the library onto Maven Local, run:

```bash
gradle -Pandroid=true publishAndroidReleasePublicationToMavenLocal
```
Expand All @@ -230,7 +230,8 @@ publish the library, containing the native binaries.

You should now be able to see the package under `~/.m2/repository/io/zenoh/zenoh-kotlin-android/1.0.0`
with the following files:
```

```raw
zenoh-kotlin-android-1.0.0-sources.jar
zenoh-kotlin-android-1.0.0.aar
zenoh-kotlin-android-1.0.0.module
Expand All @@ -241,22 +242,24 @@ Now the library is published on maven local, let's now see how to import it into

First, we need to indicate we want to look into mavenLocal for our library, so in your top level `build.gradle.kts` you need to specify
the `mavenLocal` repository:
```

```kotlin
repositories {
mavenCentral()
...
mavenLocal() // We add this line
}
```

Then in your app's `build.gradle.kts` filen add the dependency:
```
Then in your app's `build.gradle.kts` file add the dependency:

```kotlin
implementation("io.zenoh:zenoh-kotlin-android:1.0.0")
```

And finally, do not forget to add the required internet permissions on your manifest!

```
```xml
<uses-permission android:name="android.permission.INTERNET" />
<uses-permission android:name="android.permission.ACCESS_NETWORK_STATE" />
```
Expand All @@ -268,6 +271,7 @@ And that was it! You can now import the code from the `io.zenoh` package and use
Because it's a Kotlin project, we use [Dokka](https://kotlinlang.org/docs/dokka-introduction.html) to generate the documentation.

In order to build it, run:

```bash
gradle zenoh-kotlin:dokkaHtml
```
Expand Down Expand Up @@ -298,7 +302,7 @@ causes the logs to appear in standard output.

The log levels are the ones from Rust: `trace`, `info`, `debug`, `error` and `warn`.

---
----

# Examples

Expand Down
21 changes: 14 additions & 7 deletions examples/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,19 +4,18 @@

## Start instructions



```bash
gradle <example> --args="<arguments>"
gradle <example> --args="<arguments>"
```

for instance

```bash
gradle ZPub --args="-h"
gradle ZPub --args="-h"
```

will return

```bash
> Task :examples:ZPub
Usage: zpub [<options>]
Expand Down Expand Up @@ -44,20 +43,21 @@ The connect and listen parameters (that are common to all the examples) accept m
For instance:

```bash
gradle ZPub --args="-l tcp/localhost:7447 -l tcp/localhost:7448 -l tcp/localhost:7449"
gradle ZPub --args="-l tcp/localhost:7447 -l tcp/localhost:7448 -l tcp/localhost:7449"
```

There is the possibility to provide a Zenoh config file as follows

```bash
gradle ZPub --args="-c path/to/config.json5"
gradle ZPub --args="-c path/to/config.json5"
```

In that case, any other provided configuration parameters through the command line interface will not be taken into consideration.

One last comment regarding Zenoh logging for the examples, remember it can be enabled through the environment variable `RUST_LOG` as follows:

```bash
RUST_LOG=<level> gradle ZPub
RUST_LOG=<level> gradle ZPub
```

where `<level>` can be either `info`, `trace`, `debug`, `warn` or `error`.
Expand All @@ -76,12 +76,15 @@ Usage:
```bash
gradle ZPub
```

or

```bash
gradle ZPub --args="-k demo/example/test -v 'hello world'"
```

### ZSub

Creates a subscriber with a key expression.
The subscriber will be notified of each put made on any key expression matching
the subscriber's key expression, and will print this notification.
Expand All @@ -91,7 +94,9 @@ Usage:
```bash
gradle ZSub
```

or

```bash
gradle ZSub --args="-k demo/example/test"
```
Expand All @@ -105,6 +110,7 @@ will receive this query and reply with paths/values that will be received by the
```bash
gradle ZGet
```

or

```bash
Expand All @@ -129,6 +135,7 @@ gradle ZPut --args="-k demo/example/put -v 'Put from Kotlin!'"
```

### ZDelete

Performs a Delete operation into a path/value into Zenoh.

Usage:
Expand Down

0 comments on commit 70b8709

Please sign in to comment.