Merge pull request #3 from dannywillems/refacto

Refactorize the code with new OCaml tools

.github/workflows/main.yml

@@ -0,0 +1,55 @@
+name: Main workflow
+
+on:
+  - pull_request
+  - push
+
+jobs:
+  lint:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v2
+      - name: Use OCaml 4.11.0
+        uses: avsm/setup-ocaml@v1
+        with:
+          ocaml-version: 4.11.0
+      - name: "Install ocamlformat.0.15.0"
+        run: opam install ocamlformat.0.15.0
+      - name: "Check lint"
+        run: opam exec -- ocamlformat --check src/*.ml*
+
+  build:
+    strategy:
+      fail-fast: false
+      matrix:
+        os:
+          - macos-latest
+          - ubuntu-latest
+          - windows-latest
+        ocaml-version:
+          - 4.11.1
+          - 4.10.2
+          - 4.09.1
+          - 4.08.1
+
+    runs-on: ${{ matrix.os }}
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v2
+
+      - name: Use OCaml ${{ matrix.ocaml-version }}
+        uses: avsm/setup-ocaml@v1
+        with:
+          ocaml-version: ${{ matrix.ocaml-version }}
+
+      - run: opam pin add cordova-plugin-inappbrowser.dev . --no-action
+
+      - run: opam depext cordova-plugin-inappbrowser --yes --with-doc --with-test
+
+      - run: opam install . --deps-only --with-doc --with-test
+
+      - run: opam exec -- dune build
+
+      - run: opam exec -- dune runtest

.gitignore

@@ -1,4 +1,2 @@
-*.cma
-*.cmo
-*.cmi
-*.ml
+_build
+_opam

README.md

@@ -9,21 +9,22 @@ Binding to
 [Example
 application](https://github.com/dannywillems/ocaml-cordova-plugin-inappbrowser-example).
 
-## What does cordova-plugin-inappbrowser do ?
+## Setup a dev environment
 
+```shell
+# create a local switch
+opam install ocamlformat.0.15.0 merlin
+opam install -y .
 ```
-This plugin provides a web browser view that displays when calling
+
+## What does cordova-plugin-inappbrowser do ?
+
+> This plugin provides a web browser view that displays when calling
 `cordova.InAppBrowser.open()`.
-```
 
 Source: [cordova-plugin-inappbrowser](https://github.com/apache/cordova-plugin-inappbrowser)
 
-## How to install and compile your project by using this plugin ?
-
-Don't forget to switch to a compiler **>= 4.03.0**.
-```Shell
-opam switch 4.03.0
-```
+## How to install and compile your project by using this plugin?
 
 You can use opam by pinning the repository with
 ```Shell
@@ -32,7 +33,7 @@ opam pin add cordova-plugin-inappbrowser https://github.com/dannywillems/ocaml-c
 
 and to compile your project, use
 ```Shell
-ocamlfind ocamlc -c -o [output_file] -package gen_js_api -package cordova-plugin-inappbrowser [...] -linkpkg [other arguments]
+dune build @install
 ```
 
 Don't forget to install the cordova plugin inappbrowser with

opam/opam → cordova-plugin-inappbrowser.opam

@@ -1,22 +1,14 @@
-opam-version: "1.2"
-name:         "ocaml-cordova-plugin-inappbrowser"
+opam-version: "2.0"
 maintainer:   "Danny Willems <[email protected]>"
-author:       "Danny Willems <[email protected]>"
-
+authors:      "Danny Willems <[email protected]>"
 homepage:     "https://github.com/dannywillems/ocaml-cordova-plugin-inappbrowser"
 bug-reports:  "https://github.com/dannywillems/ocaml-cordova-plugin-inappbrowser/issues"
-
 dev-repo:     "https://github.com/dannywillems/ocaml-cordova-plugin-inappbrowser"
-
 license:      "LGPL-3.0 with OCaml linking exception"
-
-version:      "1.0"
-
-build: [make "build"]
-install: [make "install"]
-remove: [make "remove"]
-
+description: "Binding to the inappbrowser cordova plugin using gen_js_api"
+synopsis: "Binding to the inappbrowser cordova plugin using gen_js_api"
+build: [[ "dune" "build" "-j" jobs "-p" name "@install" ]]
 depends: [
+  "ocaml" { >= "4.03.0" }
   "gen_js_api"
 ]
-available: [ ocaml-version >= "4.03.0" ]