Allow preview browsers in the data structure

mdn · Aug 17, 2021 · d250f27 · d250f27
1 parent 13efd0c
commit d250f27
Show file tree

Hide file tree

Showing 8 changed files with 40 additions and 0 deletions.
diff --git a/browsers/chrome.json b/browsers/chrome.json
@@ -2,6 +2,7 @@
   "browsers": {
     "chrome": {
       "name": "Chrome",
+      "preview_name": "Canary",
       "pref_url": "chrome://flags",
       "releases": {
         "1": {

diff --git a/browsers/firefox.json b/browsers/firefox.json
@@ -2,6 +2,7 @@
   "browsers": {
     "firefox": {
       "name": "Firefox",
+      "preview_name": "Nightly",
       "pref_url": "about:config",
       "releases": {
         "1": {

diff --git a/browsers/safari.json b/browsers/safari.json
@@ -2,6 +2,7 @@
   "browsers": {
     "safari": {
       "name": "Safari",
+      "preview_name": "TP",
       "releases": {
         "1": {
           "release_date": "2003-06-23",

diff --git a/schemas/browsers-schema.md b/schemas/browsers-schema.md
@@ -17,6 +17,7 @@ The file `firefox.json` is structured like this:
   "browsers": {
     "firefox": {
       "name": "Firefox",
+      "preview_name": "Nightly",
       "pref_url": "about:config",
       "releases": {
         "1.5": {
@@ -48,6 +49,10 @@ An optional boolean indicating whether the browser supports flags. This is a hin
 
 An optional string containing the URL of the page where feature flags can be changed (e.g. `"about:config"` for Firefox or `"chrome://flags"` for Chrome).
 
+### `preview_name`
+
+An optional string containing the name of the preview browser. For example, "Nightly" for Firefox, "Canary" for Chrome, and "TP" for Safari.
+
 ### Release objects
 
 The release objects consist of the following properties:

diff --git a/schemas/browsers.schema.json b/schemas/browsers.schema.json
@@ -38,6 +38,10 @@
           "type": "string",
           "description": "URL of the page where feature flags can be changed (e.g. 'about:config' or 'chrome://flags')."
         },
+        "preview_name": {
+          "type": "string",
+          "description": "Preview name, avoid long-form names (use 'Nightly' instead of 'Firefox Nightly')."
+        },
         "releases": {
           "type": "object",
           "additionalProperties": { "$ref": "#/definitions/release_statement" }

diff --git a/test/linter/test-consistency.js b/test/linter/test-consistency.js
@@ -322,6 +322,15 @@ class ConsistencyChecker {
       if (b_version_added.startsWith('≤')) {
         return false;
       }
+      if (a_version_added === 'preview' && b_version_added === 'preview') {
+        return false;
+      }
+      if (b_version_added === 'preview') {
+        return true;
+      }
+      if (a_version_added === 'preview') {
+        return false;
+      }
       return compareVersions.compare(
         a_version_added.replace('≤', ''),
         b_version_added,

diff --git a/test/linter/test-versions.js b/test/linter/test-versions.js
@@ -33,6 +33,9 @@ for (const browser of Object.keys(browsers)) {
   if (VERSION_RANGE_BROWSERS[browser]) {
     validBrowserVersions[browser].push(...VERSION_RANGE_BROWSERS[browser]);
   }
+  if (browsers[browser].preview_name) {
+    validBrowserVersions[browser].push('preview');
+  }
 }
 
 /**
@@ -63,6 +66,16 @@ function addedBeforeRemoved(statement) {
     return null;
   }
 
+  if (added === 'preview' && removed === 'preview') {
+    return false;
+  }
+  if (added === 'preview' && removed !== 'preview') {
+    return false;
+  }
+  if (added !== 'preview' && removed === 'preview') {
+    return true;
+  }
+
   return compareVersions.compare(added, removed, '<');
 }
 

diff --git a/types.d.ts b/types.d.ts
@@ -45,6 +45,12 @@ export interface BrowserStatement {
    */
   name: string;
 
+  /**
+   * The preview browser's name, for example:
+   * `"Nightly"`, `"Canary"`, `"TP"`, etc.
+   */
+  preview_name: string;
+
   /**
    * The known versions of this browser.
    */