Merge pull request #627 from projectfluent/updates

Update dependencies

.github/workflows/test.yml

@@ -10,11 +10,11 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        node: [16, 18, latest]
+        node: [18, 20, latest]
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
       - name: Use Node.js ${{matrix.node}}
-        uses: actions/setup-node@v3
+        uses: actions/setup-node@v4
         with:
           node-version: ${{matrix.node}}
       - run: npm ci

fluent-bundle/README.md

@@ -47,17 +47,8 @@ The API reference is available at https://projectfluent.org/fluent.js/bundle.
 
 - `Intl.DateTimeFormat` (standard, well-supported)
 - `Intl.NumberFormat` (standard, well-supported)
-- `Intl.PluralRules` (standard, new in ECMAScript 2018)
-
-`Intl.PluralRules` may already be available in some engines. In most cases,
-however, a polyfill will be required. We recommend [intl-pluralrules][].
-
-```javascript
-import "intl-pluralrules";
-import { FluentBundle } from "@fluent/bundle";
-```
+- `Intl.PluralRules` (standard, well-supported)
 
 See also the [Compatibility][] article on the `fluent.js` wiki.
 
-[intl-pluralrules]: https://www.npmjs.com/package/intl-pluralrules
 [compatibility]: https://github.com/projectfluent/fluent.js/wiki/Compatibility

fluent-bundle/package.json

@@ -45,7 +45,7 @@
     "test": "mocha 'test/*_test.js'"
   },
   "engines": {
-    "node": ">=14.0.0",
+    "node": ">=18.0.0",
     "npm": ">=7.0.0"
   },
   "devDependencies": {

fluent-bundle/src/resource.ts

@@ -542,5 +542,8 @@ export class FluentResource {
 }
 
 class Indent {
-  constructor(public value: string, public length: number) {}
+  constructor(
+    public value: string,
+    public length: number
+  ) {}
 }

fluent-dedent/package.json

@@ -30,7 +30,7 @@
     "test": "mocha 'test/*_test.js'"
   },
   "engines": {
-    "node": ">=14.0.0",
+    "node": ">=18.0.0",
     "npm": ">=7.0.0"
   }
 }

fluent-dom/package.json

@@ -40,12 +40,12 @@
     "test": "c8 mocha -ui tdd --require ./test/index.js 'test/*_test.js'"
   },
   "engines": {
-    "node": ">=14.0.0",
+    "node": ">=18.0.0",
     "npm": ">=7.0.0"
   },
   "devDependencies": {
     "@fluent/bundle": "^0.18.0",
-    "jsdom": "^21.1.0"
+    "jsdom": "^24.0.0"
   },
   "dependencies": {
     "cached-iterable": "^0.3"

fluent-dom/test/index.js

@@ -10,6 +10,6 @@ Object.keys(document.defaultView).forEach(property => {
   }
 });
 
-global.navigator = {
-  userAgent: "node.js",
-};
+Object.defineProperty(global, "navigator", {
+  get: () => ({ userAgent: "node.js" }),
+});

fluent-langneg/package.json

@@ -38,7 +38,7 @@
     "test": "mocha 'test/*_test.js'"
   },
   "engines": {
-    "node": ">=14.0.0",
+    "node": ">=18.0.0",
     "npm": ">=7.0.0"
   }
 }

fluent-react/README.md

@@ -22,7 +22,7 @@ maximum flexibility, `@fluent/react` expects the developer to write a little
 bit of a setup code related to language negotiation and translation fetching.
 It makes `@fluent/react` unopinionated and suitable for many different
 scenarios. You will likely also need to install a few other packages:
-`@fluent/bundle`, `@fluent/langneg` and `intl-pluralrules`.
+`@fluent/bundle` and `@fluent/langneg`.
 
 Consult the [wiki][] for documentation on how to set up and use
 `@fluent/react`.

fluent-react/example/.gitignore

@@ -1,3 +1,4 @@
 .parcel-cache/
 dist/
 *.tgz
+package-lock.json