Skip to content

Commit

Permalink
fix: remove unnecessary console logs
Browse files Browse the repository at this point in the history
  • Loading branch information
jsnwesson committed Feb 9, 2024
1 parent 8f062be commit cec5f45
Showing 1 changed file with 5 additions and 8 deletions.
13 changes: 5 additions & 8 deletions patches/@edx+frontend-build+13.0.14.patch
Original file line number Diff line number Diff line change
@@ -1,12 +1,10 @@
diff --git a/node_modules/@edx/frontend-build/config/jest.config.js b/node_modules/@edx/frontend-build/config/jest.config.js
index ae08b70..2c8c0fa 100644
index ae08b70..9440d81 100644
--- a/node_modules/@edx/frontend-build/config/jest.config.js
+++ b/node_modules/@edx/frontend-build/config/jest.config.js
@@ -3,12 +3,16 @@ const fs = require('fs');

@@ -4,11 +4,14 @@ const fs = require('fs');
const presets = require('../lib/presets');

+console.log('If you are expecting to use an env.config.jsx config, make sure to run `npm run postinstall` first!')
let envConfigPath = path.resolve(__dirname, './jest/fallback.env.config.js');
-const appEnvConfigPath = path.resolve(process.cwd(), './env.config.js');
+const appEnvConfigPathJs = path.resolve(process.cwd(), './env.config.js');
Expand All @@ -24,7 +22,7 @@ index ae08b70..2c8c0fa 100644
module.exports = {
testURL: 'http://localhost/',
diff --git a/node_modules/@edx/frontend-build/config/webpack.dev.config.js b/node_modules/@edx/frontend-build/config/webpack.dev.config.js
index 5ce7716..6828971 100644
index 5ce7716..fe9888e 100644
--- a/node_modules/@edx/frontend-build/config/webpack.dev.config.js
+++ b/node_modules/@edx/frontend-build/config/webpack.dev.config.js
@@ -7,6 +7,7 @@ const Dotenv = require('dotenv-webpack');
Expand All @@ -35,11 +33,10 @@ index 5ce7716..6828971 100644
const PostCssAutoprefixerPlugin = require('autoprefixer');
const PostCssRTLCSS = require('postcss-rtlcss');
const PostCssCustomMediaCSS = require('postcss-custom-media');
@@ -17,6 +18,17 @@ const presets = require('../lib/presets');
@@ -17,6 +18,16 @@ const presets = require('../lib/presets');
const resolvePrivateEnvConfig = require('../lib/resolvePrivateEnvConfig');
const getLocalAliases = require('./getLocalAliases');

+console.log('If you are expecting to use the PORT from env.config, make sure to run `npm run postinstall` first!');
+let envConfig = {};
+const envConfigPathJs = path.resolve(process.cwd(),'./env.config.js');
+const envConfigPathJsx = path.resolve(process.cwd(), './env.config.jsx');
Expand All @@ -53,7 +50,7 @@ index 5ce7716..6828971 100644
// Add process env vars. Currently used only for setting the
// server port and the publicPath
dotenv.config({
@@ -174,7 +186,7 @@ module.exports = merge(commonConfig, {
@@ -174,7 +185,7 @@ module.exports = merge(commonConfig, {
// reloading.
devServer: {
host: '0.0.0.0',
Expand Down

0 comments on commit cec5f45

Please sign in to comment.