fix(config): remove option that does not serialise as JSON

Author: thibaudcolas

src/__snapshots__/semver.test.js.snap

@@ -22,9 +22,6 @@ Object {
     "id-length": Array [
       0,
     ],
-    "import/no-cycle": Array [
-      2,
-    ],
     "import/no-extraneous-dependencies": Array [
       2,
       Object {

src/base.js

@@ -46,9 +46,6 @@ module.exports = {
     // Does not work at the moment for nested fields.
     // https://github.com/evcohen/eslint-plugin-jsx-a11y/blob/master/docs/rules/label-has-for.md
     "jsx-a11y/label-has-for": [0],
-    // Override airbnb config to make it serialisable to JSON.
-    // https://github.com/airbnb/javascript/blob/32a7bf44c2e5b3d2a5d05af46716fbe45bc09bec/packages/eslint-config-airbnb-base/rules/imports.js#L230
-    "import/no-cycle": [2],
   },
 
   // http://eslint.org/docs/user-guide/configuring#specifying-environments

src/config.js

@@ -15,6 +15,9 @@ const getComputedConfig = (baseConfig) => {
   delete config.filePath;
   delete config.baseDirectory;
 
+  // Remove option that does not serialise well to JSON.
+  config.rules["import/no-cycle"] = config.rules["import/no-cycle"][0];
+
   return config;
 };