From a2af79528ca7225bd68548c2a3ba27b833ae421c Mon Sep 17 00:00:00 2001 From: Logan McAnsh Date: Thu, 9 Mar 2023 13:53:17 -0500 Subject: [PATCH] chore: remove default ignoredRouteFiles Signed-off-by: Logan McAnsh --- docs/file-conventions/remix-config.md | 4 +--- docs/guides/migrating-react-router-app.md | 1 - integration/flat-routes-test.ts | 2 +- integration/helpers/cf-template/remix.config.js | 1 - integration/helpers/deno-template/remix.config.js | 1 - integration/helpers/node-template/remix.config.js | 1 - integration/meta-test.ts | 1 - .../remix-dev/__tests__/fixtures/cloudflare/remix.config.js | 1 - packages/remix-dev/__tests__/fixtures/deno/remix.config.js | 1 - packages/remix-dev/__tests__/fixtures/node/remix.config.js | 1 - packages/remix-dev/__tests__/fixtures/stack/remix.config.js | 1 - templates/arc/remix.config.js | 1 - templates/cloudflare-pages/remix.config.js | 1 - templates/cloudflare-workers/remix.config.js | 1 - templates/deno/remix.config.js | 1 - templates/express/remix.config.js | 1 - templates/fly/remix.config.js | 1 - templates/netlify/remix.config.js | 1 - templates/remix/remix.config.js | 1 - templates/vercel/remix.config.js | 1 - 20 files changed, 2 insertions(+), 22 deletions(-) diff --git a/docs/file-conventions/remix-config.md b/docs/file-conventions/remix-config.md index f2b4e5e54c1..7d9177107c6 100644 --- a/docs/file-conventions/remix-config.md +++ b/docs/file-conventions/remix-config.md @@ -11,7 +11,6 @@ This file has a few build and development configuration options, but does not ac module.exports = { appDirectory: "app", assetsBuildDirectory: "public/build", - ignoredRouteFiles: ["**/.*"], publicPath: "/build/", routes(defineRoutes) { return defineRoutes((route) => { @@ -164,14 +163,13 @@ a `@sindresorhus/slugify` which is ESM-only as well. Here's how you would be able to consume those packages in a CJS app without having to use dynamic imports: -```ts filename=remix.config.js lines=[8-13] +```ts filename=remix.config.js lines=[7-12] /** @type {import('@remix-run/dev').AppConfig} */ module.exports = { appDirectory: "app", assetsBuildDirectory: "public/build", publicPath: "/build/", serverBuildDirectory: "build", - ignoredRouteFiles: ["**/.*"], serverDependenciesToBundle: [ /^rehype.*/, /^remark.*/, diff --git a/docs/guides/migrating-react-router-app.md b/docs/guides/migrating-react-router-app.md index 4cb2d46c004..c28b2a16350 100644 --- a/docs/guides/migrating-react-router-app.md +++ b/docs/guides/migrating-react-router-app.md @@ -335,7 +335,6 @@ Every Remix app accepts a `remix.config.js` file in the project root. While its /** @type {import('@remix-run/dev').AppConfig} */ module.exports = { appDirectory: "app", - ignoredRouteFiles: ["**/.*"], assetsBuildDirectory: "public/build", }; ``` diff --git a/integration/flat-routes-test.ts b/integration/flat-routes-test.ts index 9254ca16ee5..cce9e96f158 100644 --- a/integration/flat-routes-test.ts +++ b/integration/flat-routes-test.ts @@ -14,7 +14,7 @@ test.describe("flat routes", () => { fixture = await createFixture({ future: { v2_routeConvention: true }, files: { - "app/root.jsx": js` + "app/root.tsx": js` import { Links, Meta, Outlet, Scripts } from "@remix-run/react"; export default function Root() { diff --git a/integration/helpers/cf-template/remix.config.js b/integration/helpers/cf-template/remix.config.js index ab7a9e7da4c..3fb5b04eb3b 100644 --- a/integration/helpers/cf-template/remix.config.js +++ b/integration/helpers/cf-template/remix.config.js @@ -1,7 +1,6 @@ /** @type {import('@remix-run/dev').AppConfig} */ module.exports = { devServerBroadcastDelay: 1000, - ignoredRouteFiles: ["**/.*"], server: "./server.js", serverConditions: ["worker"], serverDependenciesToBundle: "all", diff --git a/integration/helpers/deno-template/remix.config.js b/integration/helpers/deno-template/remix.config.js index 4ad663f5b71..0d83af8b690 100644 --- a/integration/helpers/deno-template/remix.config.js +++ b/integration/helpers/deno-template/remix.config.js @@ -7,7 +7,6 @@ module.exports = { If live reload seems slow, try to decrease the dev server broadcast delay. */ devServerBroadcastDelay: 300, - ignoredRouteFiles: ["**/.*"], server: "./server.ts", serverConditions: ["deno", "worker"], serverDependenciesToBundle: "all", diff --git a/integration/helpers/node-template/remix.config.js b/integration/helpers/node-template/remix.config.js index b7f693265aa..e0fde67d446 100644 --- a/integration/helpers/node-template/remix.config.js +++ b/integration/helpers/node-template/remix.config.js @@ -1,6 +1,5 @@ /** @type {import('@remix-run/dev').AppConfig} */ module.exports = { - ignoredRouteFiles: ["**/.*"], // appDirectory: "app", // assetsBuildDirectory: "public/build", // serverBuildPath: "build/index.js", diff --git a/integration/meta-test.ts b/integration/meta-test.ts index 5301a8f2870..53ef022f3d7 100644 --- a/integration/meta-test.ts +++ b/integration/meta-test.ts @@ -399,7 +399,6 @@ test.describe("v2_meta", () => { files: { "remix.config.js": js` module.exports = { - ignoredRouteFiles: ["**/.*"], future: { v2_meta: true, }, diff --git a/packages/remix-dev/__tests__/fixtures/cloudflare/remix.config.js b/packages/remix-dev/__tests__/fixtures/cloudflare/remix.config.js index 7d8260a36a7..96dc8387fa3 100644 --- a/packages/remix-dev/__tests__/fixtures/cloudflare/remix.config.js +++ b/packages/remix-dev/__tests__/fixtures/cloudflare/remix.config.js @@ -3,7 +3,6 @@ module.exports = { serverBuildTarget: "cloudflare-pages", server: "./server.js", devServerBroadcastDelay: 1000, - ignoredRouteFiles: ["**/.*"], // appDirectory: "app", // assetsBuildDirectory: "public/build", // serverBuildPath: "functions/[[path]].js", diff --git a/packages/remix-dev/__tests__/fixtures/deno/remix.config.js b/packages/remix-dev/__tests__/fixtures/deno/remix.config.js index d621422e56f..4b0f4515c22 100644 --- a/packages/remix-dev/__tests__/fixtures/deno/remix.config.js +++ b/packages/remix-dev/__tests__/fixtures/deno/remix.config.js @@ -9,7 +9,6 @@ module.exports = { If live reload seems slow, try to decrease the dev server broadcast delay. */ devServerBroadcastDelay: 300, - ignoredRouteFiles: ["**/.*"], // appDirectory: "app", // assetsBuildDirectory: "public/build", // serverBuildPath: "build/index.js", diff --git a/packages/remix-dev/__tests__/fixtures/node/remix.config.js b/packages/remix-dev/__tests__/fixtures/node/remix.config.js index adf2a0b5d3e..9d355b29b38 100644 --- a/packages/remix-dev/__tests__/fixtures/node/remix.config.js +++ b/packages/remix-dev/__tests__/fixtures/node/remix.config.js @@ -1,6 +1,5 @@ /** @type {import('@remix-run/dev').AppConfig} */ module.exports = { - ignoredRouteFiles: ["**/.*"], // appDirectory: "app", // assetsBuildDirectory: "public/build", // serverBuildPath: "build/index.js", diff --git a/packages/remix-dev/__tests__/fixtures/stack/remix.config.js b/packages/remix-dev/__tests__/fixtures/stack/remix.config.js index 5bb0822e720..4c5e7f737a7 100644 --- a/packages/remix-dev/__tests__/fixtures/stack/remix.config.js +++ b/packages/remix-dev/__tests__/fixtures/stack/remix.config.js @@ -1,7 +1,6 @@ /** @type {import('@remix-run/dev').AppConfig} */ module.exports = { serverBuildTarget: "node-cjs", - ignoredRouteFiles: ["**/.*"], // appDirectory: "app", // assetsBuildDirectory: "public/build", // serverBuildPath: "build/index.js", diff --git a/templates/arc/remix.config.js b/templates/arc/remix.config.js index a42ab1855f3..0e063fb0429 100644 --- a/templates/arc/remix.config.js +++ b/templates/arc/remix.config.js @@ -1,6 +1,5 @@ /** @type {import('@remix-run/dev').AppConfig} */ module.exports = { - ignoredRouteFiles: ["**/.*"], publicPath: "/_static/build/", server: "./server.js", serverBuildPath: "server/index.js", diff --git a/templates/cloudflare-pages/remix.config.js b/templates/cloudflare-pages/remix.config.js index 1a0ec28bf46..cc79172f4a8 100644 --- a/templates/cloudflare-pages/remix.config.js +++ b/templates/cloudflare-pages/remix.config.js @@ -1,7 +1,6 @@ /** @type {import('@remix-run/dev').AppConfig} */ module.exports = { devServerBroadcastDelay: 1000, - ignoredRouteFiles: ["**/.*"], server: "./server.js", serverBuildPath: "functions/[[path]].js", serverConditions: ["worker"], diff --git a/templates/cloudflare-workers/remix.config.js b/templates/cloudflare-workers/remix.config.js index ab7a9e7da4c..3fb5b04eb3b 100644 --- a/templates/cloudflare-workers/remix.config.js +++ b/templates/cloudflare-workers/remix.config.js @@ -1,7 +1,6 @@ /** @type {import('@remix-run/dev').AppConfig} */ module.exports = { devServerBroadcastDelay: 1000, - ignoredRouteFiles: ["**/.*"], server: "./server.js", serverConditions: ["worker"], serverDependenciesToBundle: "all", diff --git a/templates/deno/remix.config.js b/templates/deno/remix.config.js index 4ad663f5b71..0d83af8b690 100644 --- a/templates/deno/remix.config.js +++ b/templates/deno/remix.config.js @@ -7,7 +7,6 @@ module.exports = { If live reload seems slow, try to decrease the dev server broadcast delay. */ devServerBroadcastDelay: 300, - ignoredRouteFiles: ["**/.*"], server: "./server.ts", serverConditions: ["deno", "worker"], serverDependenciesToBundle: "all", diff --git a/templates/express/remix.config.js b/templates/express/remix.config.js index adf2a0b5d3e..9d355b29b38 100644 --- a/templates/express/remix.config.js +++ b/templates/express/remix.config.js @@ -1,6 +1,5 @@ /** @type {import('@remix-run/dev').AppConfig} */ module.exports = { - ignoredRouteFiles: ["**/.*"], // appDirectory: "app", // assetsBuildDirectory: "public/build", // serverBuildPath: "build/index.js", diff --git a/templates/fly/remix.config.js b/templates/fly/remix.config.js index adf2a0b5d3e..9d355b29b38 100644 --- a/templates/fly/remix.config.js +++ b/templates/fly/remix.config.js @@ -1,6 +1,5 @@ /** @type {import('@remix-run/dev').AppConfig} */ module.exports = { - ignoredRouteFiles: ["**/.*"], // appDirectory: "app", // assetsBuildDirectory: "public/build", // serverBuildPath: "build/index.js", diff --git a/templates/netlify/remix.config.js b/templates/netlify/remix.config.js index 167e139f331..11fba5aa8c4 100644 --- a/templates/netlify/remix.config.js +++ b/templates/netlify/remix.config.js @@ -1,6 +1,5 @@ /** @type {import('@remix-run/dev').AppConfig} */ module.exports = { - ignoredRouteFiles: ["**/.*"], server: process.env.NETLIFY || process.env.NETLIFY_LOCAL ? "./server.js" diff --git a/templates/remix/remix.config.js b/templates/remix/remix.config.js index adf2a0b5d3e..9d355b29b38 100644 --- a/templates/remix/remix.config.js +++ b/templates/remix/remix.config.js @@ -1,6 +1,5 @@ /** @type {import('@remix-run/dev').AppConfig} */ module.exports = { - ignoredRouteFiles: ["**/.*"], // appDirectory: "app", // assetsBuildDirectory: "public/build", // serverBuildPath: "build/index.js", diff --git a/templates/vercel/remix.config.js b/templates/vercel/remix.config.js index 3add2266a50..6d367a4b8c1 100644 --- a/templates/vercel/remix.config.js +++ b/templates/vercel/remix.config.js @@ -1,6 +1,5 @@ /** @type {import('@remix-run/dev').AppConfig} */ module.exports = { - ignoredRouteFiles: ["**/.*"], // When running locally in development mode, we use the built-in remix // server. This does not understand the vercel lambda module format, // so we default back to the standard build output.