Skip to content
This repository was archived by the owner on May 10, 2021. It is now read-only.

Commit 802d3eb

Browse files
committed
Refactor: Move build steps into lib/steps and helpers into lib/helpers
Clean up the list of files under lib by grouping them into build steps and helper files. Build steps are the functions that are directly called from the next-on-netlify.js file at root level. Helpers are auxiliary functions used in one of the build steps.
1 parent ee833ce commit 802d3eb

13 files changed

+31
-31
lines changed

lib/config.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
const { join } = require("path");
2-
const getNextDistDir = require("./getNextDistDir");
2+
const getNextDistDir = require("./helpers/getNextDistDir");
33

44
// This is where next-on-netlify will place all static files.
55
// The publish key in netlify.toml should point to this folder.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.

lib/copyNextAssets.js renamed to lib/steps/copyNextAssets.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
const { join } = require("path");
22
const { copySync } = require("fs-extra");
3-
const { logTitle } = require("./logger");
4-
const { NETLIFY_PUBLISH_PATH, NEXT_DIST_DIR } = require("./config");
3+
const { logTitle } = require("../helpers/logger");
4+
const { NETLIFY_PUBLISH_PATH, NEXT_DIST_DIR } = require("../config");
55

66
// Copy the NextJS' static assets from NextJS distDir to Netlify publish folder.
77
// These need to be available for NextJS to work.

lib/copyPublicFiles.js renamed to lib/steps/copyPublicFiles.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
const { existsSync, copySync } = require("fs-extra");
2-
const { logTitle } = require("./logger");
3-
const { NETLIFY_PUBLISH_PATH, PUBLIC_PATH } = require("./config");
2+
const { logTitle } = require("../helpers/logger");
3+
const { NETLIFY_PUBLISH_PATH, PUBLIC_PATH } = require("../config");
44

55
// Copy files from public folder to Netlify publish folder
66
const copyPublicFiles = () => {

lib/prepareFolders.js renamed to lib/steps/prepareFolders.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
const { emptyDirSync } = require("fs-extra");
2-
const { logTitle, log } = require("./logger");
3-
const { NETLIFY_PUBLISH_PATH, NETLIFY_FUNCTIONS_PATH } = require("./config");
2+
const { logTitle, log } = require("../helpers/logger");
3+
const { NETLIFY_PUBLISH_PATH, NETLIFY_FUNCTIONS_PATH } = require("../config");
44

55
// Empty existing publish and functions folders
66
const prepareFolders = () => {

lib/setupHtmlPages.js renamed to lib/steps/setupHtmlPages.js

+3-3
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
11
const path = require("path");
22
const { join } = path;
33
const { copySync, existsSync } = require("fs-extra");
4-
const { logTitle, logItem } = require("./logger");
5-
const { NEXT_DIST_DIR, NETLIFY_PUBLISH_PATH } = require("./config");
6-
const allNextJsPages = require("./allNextJsPages");
4+
const { logTitle, logItem } = require("../helpers/logger");
5+
const { NEXT_DIST_DIR, NETLIFY_PUBLISH_PATH } = require("../config");
6+
const allNextJsPages = require("../allNextJsPages");
77

88
// Identify all pages that have been pre-rendered and copy each one to the
99
// Netlify publish directory.

lib/setupRedirects.js renamed to lib/steps/setupRedirects.js

+5-5
Original file line numberDiff line numberDiff line change
@@ -7,11 +7,11 @@ const {
77
const {
88
getSortedRoutes,
99
} = require("@sls-next/lambda-at-edge/dist/lib/sortedRoutes");
10-
const { logTitle, logItem } = require("./logger");
11-
const { NETLIFY_PUBLISH_PATH, CUSTOM_REDIRECTS_PATH } = require("./config");
12-
const allNextJsPages = require("./allNextJsPages");
13-
const getNetlifyRoute = require("./getNetlifyRoute");
14-
const getNetlifyFunctionName = require("./getNetlifyFunctionName");
10+
const { logTitle, logItem } = require("../helpers/logger");
11+
const { NETLIFY_PUBLISH_PATH, CUSTOM_REDIRECTS_PATH } = require("../config");
12+
const allNextJsPages = require("../allNextJsPages");
13+
const getNetlifyRoute = require("../helpers/getNetlifyRoute");
14+
const getNetlifyFunctionName = require("../helpers/getNetlifyFunctionName");
1515

1616
// Setup _redirects file that routes all requests to the appropriate location:
1717
// Either the relevant Netlify function or one of the pre-rendered HTML pages.

lib/setupSsgPages.js renamed to lib/steps/setupSsgPages.js

+4-4
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,15 @@
11
const path = require("path");
22
const { join } = path;
33
const { copySync, existsSync } = require("fs-extra");
4-
const { logTitle, logItem } = require("./logger");
4+
const { logTitle, logItem } = require("../helpers/logger");
55
const {
66
NEXT_DIST_DIR,
77
NETLIFY_PUBLISH_PATH,
88
NETLIFY_FUNCTIONS_PATH,
99
FUNCTION_TEMPLATE_PATH,
10-
} = require("./config");
11-
const allNextJsPages = require("./allNextJsPages");
12-
const getNetlifyFunctionName = require("./getNetlifyFunctionName");
10+
} = require("../config");
11+
const allNextJsPages = require("../allNextJsPages");
12+
const getNetlifyFunctionName = require("../helpers/getNetlifyFunctionName");
1313

1414
// Identify all pages that require server-side rendering and create a separate
1515
// Netlify Function for every page.

lib/setupSsrPages.js renamed to lib/steps/setupSsrPages.js

+4-4
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,14 @@
11
const path = require("path");
22
const { join } = path;
33
const { copySync, existsSync } = require("fs-extra");
4-
const { logTitle, logItem } = require("./logger");
4+
const { logTitle, logItem } = require("../helpers/logger");
55
const {
66
NEXT_DIST_DIR,
77
NETLIFY_FUNCTIONS_PATH,
88
FUNCTION_TEMPLATE_PATH,
9-
} = require("./config");
10-
const allNextJsPages = require("./allNextJsPages");
11-
const getNetlifyFunctionName = require("./getNetlifyFunctionName");
9+
} = require("../config");
10+
const allNextJsPages = require("../allNextJsPages");
11+
const getNetlifyFunctionName = require("../helpers/getNetlifyFunctionName");
1212

1313
// Identify all pages that require server-side rendering and create a separate
1414
// Netlify Function for every page.

next-on-netlify.js

+8-8
Original file line numberDiff line numberDiff line change
@@ -9,14 +9,14 @@ program
99
)
1010
.parse(process.argv);
1111

12-
const { logTitle } = require("./lib/logger");
13-
const prepareFolders = require("./lib/prepareFolders");
14-
const copyPublicFiles = require("./lib/copyPublicFiles");
15-
const copyNextAssets = require("./lib/copyNextAssets");
16-
const setupSsrPages = require("./lib/setupSsrPages");
17-
const setupSsgPages = require("./lib/setupSsgPages");
18-
const setupHtmlPages = require("./lib/setupHtmlPages");
19-
const setupRedirects = require("./lib/setupRedirects");
12+
const { logTitle } = require("./lib/helpers/logger");
13+
const prepareFolders = require("./lib/steps/prepareFolders");
14+
const copyPublicFiles = require("./lib/steps/copyPublicFiles");
15+
const copyNextAssets = require("./lib/steps/copyNextAssets");
16+
const setupSsrPages = require("./lib/steps/setupSsrPages");
17+
const setupSsgPages = require("./lib/steps/setupSsgPages");
18+
const setupHtmlPages = require("./lib/steps/setupHtmlPages");
19+
const setupRedirects = require("./lib/steps/setupRedirects");
2020

2121
prepareFolders();
2222

0 commit comments

Comments
 (0)