Use ES modules, add types and github action checks

Author: ahocevar

Diff for: .circleci/config.yml

@@ -0,0 +1,21 @@
+name: Node.js CI
+
+on:
+  push:
+    branches: [ master ]
+  pull_request:
+    branches: [ master ]
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+
+    steps:
+    - uses: actions/checkout@v3
+    - name: Setup Node.js
+      uses: actions/setup-node@v3
+      with:
+        node-version: '20.x'
+    - run: npm ci
+    - run: npm run build --if-present
+    - run: npm test

Diff for: .github/workflows/pullrequest.yml

@@ -1 +1,2 @@
 node_modules
+**/*.d.ts

Diff for: index.d.ts.map

@@ -29,7 +29,13 @@ var italicRE = /(italic|oblique)$/i;
 
 var fontCache = {};
 
-module.exports = function(fonts, size, lineHeight) {
+/**
+ * @param {string|Array<string>} fonts Mapbox GL Style fontstack or single font, e.g. `['Open Sans Regular', 'Arial Unicode MS Regular']` or `'Open Sans Regular'`.
+ * @param {number} size Font size in pixels.
+ * @param {string|number} [lineHeight] Line height as css line-height.
+ * @returns {string} CSS font definition, e.g. `'normal 400 16px/1.2 "Open Sans"'`.
+ */
+export default function(fonts, size, lineHeight) {
   var cssData = fontCache[fonts];
   if (!cssData) {
     if (!Array.isArray(fonts)) {
@@ -79,4 +85,4 @@ module.exports = function(fonts, size, lineHeight) {
     cssData = fontCache[fonts] = [style, weight, fontFamilies];
   }
   return cssData[0] + sp + cssData[1] + sp + size + 'px' + (lineHeight ? '/' + lineHeight : '') + sp + cssData[2];
-};
+}

Diff for: index.js

@@ -1,8 +1,10 @@
 {
   "name": "mapbox-to-css-font",
-  "version": "2.4.5",
+  "version": "3.0.0",
   "description": "Utility to convert Mapbox GL Style font names to CSS font definitions",
-  "main": "index.js",
+  "type": "module",
+  "module": "index.js",
+  "types": "index.d.ts",
   "repository": {
     "type": "git",
     "url": "git://github.com/openlayers/mapbox-to-css-font.git"
@@ -15,15 +17,18 @@
   "license": "BSD-2-Clause",
   "scripts": {
     "lint": "eslint *.js test",
-    "pretest": "npm run lint",
-    "test": "mocha test/index.test.js"
+    "typecheck": "tsc --noEmit -p tsconfig.build.json",
+    "pretest": "npm run lint && npm run typecheck",
+    "test": "mocha test/index.test.js",
+    "prepare": "tsc --emitDeclarationOnly -p tsconfig.build.json"
   },
   "devDependencies": {
     "eslint": "^5.15.0",
     "eslint-config-openlayers": "^11.0.0",
     "karma-mocha": "^2.0.1",
     "mocha": "^10.2.0",
     "should": "^13.2.3",
-    "sinon": "^7.2.7"
+    "sinon": "^7.2.7",
+    "typescript": "^5.5.3"
   }
 }

Diff for: package-lock.json

@@ -1,6 +1,6 @@
 /* eslint-env mocha */
-const should = require('should');
-const parseFont = require('../index.js');
+import should from 'should';
+import parseFont from '../index.js';
 
 describe('mapbox-to-css-font', function() {
 

Diff for: package.json

@@ -0,0 +1,11 @@
+{
+  "compilerOptions": {
+    "allowJs": true,
+    "checkJs": true,
+    "lib": ["es2015"],
+    "declaration": true,
+    "declarationMap": true
+  },
+  "include": ["**/*.js"],
+  "exclude": ["node_modules", "test"]
+}