Fix: Gatsby-Plugin-Offline Double Reload Issue

.circleci/config.yml

@@ -139,15 +139,10 @@ commands:
         default: ""
     steps:
       - checkout
-      # In case of failure, add these steps again. Cache probably got deleted
-      #- <<: *restore_cache
-      #- <<: *install_node_modules
-      #- <<: *persist_cache
       - unless:
           condition: << parameters.skip_file_change_test >>
           steps:
             - run: ./scripts/assert-changed-files.sh "<< parameters.trigger_pattern >>|<< parameters.test_path >>/*"
-
       - when:
           condition: << parameters.slices >>
           steps:

.cursorignore

@@ -0,0 +1 @@
+# Add directories or file patterns to ignore during indexing (e.g. foo/ or *.csv)

.eslintrc.js

@@ -120,16 +120,10 @@ module.exports = {
       plugins: [`@typescript-eslint/eslint-plugin`],
       extends: [`plugin:@typescript-eslint/recommended`],
       rules: {
-        // We should absolutely avoid using ts-ignore, but it's not always possible.
-        // particular when a dependencies types are incorrect.
         "@typescript-eslint/ban-ts-comment": [
           `warn`,
           { "ts-ignore": `allow-with-description` },
         ],
-        // This rule is great. It helps us not throw on types for areas that are
-        // easily inferrable. However we have a desire to have all function inputs
-        // and outputs declaratively typed. So this let's us ignore the parameters
-        // inferrable lint.
         "@typescript-eslint/no-inferrable-types": [
           `error`,
           { ignoreParameters: true },
@@ -149,7 +143,6 @@ module.exports = {
           },
         ],
         camelcase: `off`,
-        // TODO: These rules allow a lot of stuff and don't really enforce. If we want to apply our styleguide, we'd need to fix a lot of stuff
         "@typescript-eslint/naming-convention": [
           `error`,
           {
@@ -207,20 +200,8 @@ module.exports = {
             prefix: [`I`],
           },
         ],
-        // This rule tries to prevent using `require()`. However in node code,
-        // there are times where this makes sense. And it specifically is causing
-        // problems in our tests where we often want this functionality for module
-        // mocking. At this point it's easier to have it off and just encourage
-        // using top-level imports via code reviews.
         "@typescript-eslint/no-var-requires": `off`,
         "@typescript-eslint/no-extra-semi": `off`,
-        // This rule ensures that typescript types do not have semicolons
-        // at the end of their lines, since our prettier setup is to have no semicolons
-        // e.g.,
-        // interface Foo {
-        // -  baz: string;
-        // +  baz: string
-        // }
         "@typescript-eslint/member-delimiter-style": [
           `error`,
           {
@@ -230,13 +211,7 @@ module.exports = {
           },
         ],
         "@typescript-eslint/no-empty-function": `off`,
-        // This ensures that we always type the return type of functions
-        // a high level focus of our TS setup is typing fn inputs and outputs.
         "@typescript-eslint/explicit-function-return-type": `error`,
-        // This forces us to use interfaces over types aliases for object definitions.
-        // Type is still useful for opaque types
-        // e.g.,
-        // type UUID = string
         "@typescript-eslint/consistent-type-definitions": [
           `error`,
           `interface`,
@@ -245,9 +220,6 @@ module.exports = {
           `error`,
           { functions: false },
         ],
-        // Allows us to write unions like `type Foo = "baz" | "bar"`
-        // otherwise eslint will want to switch the strings to backticks,
-        // which then crashes the ts compiler
         quotes: `off`,
         "@typescript-eslint/quotes": [
           2,
@@ -256,8 +228,6 @@ module.exports = {
             avoidEscape: true,
           },
         ],
-        // bump to @typescript-eslint/parser started showing Flow related errors in ts(x) files
-        // so disabling them in .ts(x) files
         "flowtype/no-types-missing-file-annotation": `off`,
         "@typescript-eslint/array-type": [`error`, { default: `generic` }],
       },